diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2019-11-12 21:53:31 +0100 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2019-11-13 15:06:42 +0100 |
| commit | 913d34ec6b8fdb2796f76ce90fee51ade2051189 (patch) | |
| tree | 261aefa505e7d71ad38c9d5b2ae0cfe92e9738c8 /avmedia | |
| parent | eea0879cdf1b40e6ce424dd97b58e2a84846ad79 (diff) | |
Extend loplugin:salbool to loplugin:fakebool
...checking for unnecessary uses of more "fake bool" types.
In the past, some of the checks involving the types of variables or data
members, or the return types of functions, issued warnings that required
surrounding code to be changed too (e.g., when changing the signature of a
function whose address was taken). These checks have been tightened now to not
warn in such cases (which avoids warnings that require changes to additional
code, or changes that might even be impossible to make, at the cost of being
less aggressive about removing all unnecessary uses of those "fake bool" types).
Change-Id: I70eb75039817cda34ed611387ee27dc5f36a3e2e
Reviewed-on: https://gerrit.libreoffice.org/82554
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'avmedia')
| -rw-r--r-- | avmedia/source/gstreamer/gstplayer.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/avmedia/source/gstreamer/gstplayer.cxx b/avmedia/source/gstreamer/gstplayer.cxx index 4612c1053998..764a27639eb5 100644 --- a/avmedia/source/gstreamer/gstplayer.cxx +++ b/avmedia/source/gstreamer/gstplayer.cxx @@ -414,7 +414,7 @@ void Player::processMessage( GstMessage *message ) #define LCL_WAYLAND_DISPLAY_HANDLE_CONTEXT_TYPE "GstWaylandDisplayHandleContextType" -static gboolean lcl_is_wayland_display_handle_need_context_message(GstMessage* msg) +static bool lcl_is_wayland_display_handle_need_context_message(GstMessage* msg) { g_return_val_if_fail(GST_IS_MESSAGE(msg), false); |