diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2018-04-16 16:48:41 +0200 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2018-04-17 07:47:57 +0200 |
| commit | 0a910746b19f10f184f6ff8f41c868994a472ca9 (patch) | |
| tree | b6d1e0260d0f02f51d1e7946fb9faff57676359f /chart2 | |
| parent | external/firebird: Better workaround for Clang alignment expectations (diff) | |
| download | core-0a910746b19f10f184f6ff8f41c868994a472ca9.tar.gz core-0a910746b19f10f184f6ff8f41c868994a472ca9.zip | |
Clarify checking o_pbLink directly, versus missing dereferencing
But given 18ab7abaa9426cd27092125637fdf5fb849b4a04
"MediaWindow::executeMediaURLDialog: add link button" documents that parameter
as "if not 0, this is an 'insert' dialog", it looks plausible that all these
places indeed want to check the pointer-to-bool for non-nullness, rather than
checking the dereferenced bool.
Change-Id: Ifbd5e4b711bee97f3ba0b6aab556f533574d21c6
Reviewed-on: https://gerrit.libreoffice.org/52992
Reviewed-by: Michael Stahl <Michael.Stahl@cib.de>
Tested-by: Jenkins <ci@libreoffice.org>
Diffstat (limited to 'chart2')
0 files changed, 0 insertions, 0 deletions