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 /vcl/CppunitTest_vcl_lifecycle.mk | |
parent | 25764ffd4db0e5db6f9cc9f3da8691e607f48b83 (diff) |
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 'vcl/CppunitTest_vcl_lifecycle.mk')
0 files changed, 0 insertions, 0 deletions