diff options
| author | Stephan Bergmann <sbergman@redhat.com> | 2014-02-24 17:25:05 +0100 |
|---|---|---|
| committer | Stephan Bergmann <sbergman@redhat.com> | 2014-02-24 17:25:05 +0100 |
| commit | 76b114f849645f42311c0553b3320532bb26049c (patch) | |
| tree | aa1ac2dc1a6306fb840b54f76f5a9cbe050de39b /starmath/source/smdetect.cxx | |
| parent | 8865a555afa6aa33614a90167cb07b5fafa8b8ed (diff) | |
implicitboolconversion: warn about implicit conversion of call args to bool
...to be able to find problems like 6e0bdf04add338b7d5b29fc7b3fc9f08cfd5e96f
"sal_Bool arg of SetUseImagesInMenus was abused to squeeze '2' through it"
earlier when converting occurrences of sal_Bool to bool.
Restricting this check to function call arguments avoids too much noise while
hopefully still catching all the relevant problems.
(This check partially overlaps the pointertobool check, so implicit conversions
from pointers to bool call arguments will now generate two loplugin warnings,
but that's harmless.)
Change-Id: I0b03b1d1615aaf8bc18e7a84c56fff3ef9903508
Diffstat (limited to 'starmath/source/smdetect.cxx')
| -rw-r--r-- | starmath/source/smdetect.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/starmath/source/smdetect.cxx b/starmath/source/smdetect.cxx index cd6a2c760475..d25f0dd54654 100644 --- a/starmath/source/smdetect.cxx +++ b/starmath/source/smdetect.cxx @@ -230,7 +230,7 @@ OUString SAL_CALL SmFilterDetect::detect( Sequence< PropertyValue >& lDescriptor OUString aTmpFilterName; if ( pFilter ) aTmpFilterName = pFilter->GetName(); - aTypeName = SfxFilter::GetTypeFromStorage( xStorage, pFilter ? pFilter->IsAllowedAsTemplate() : sal_False, &aTmpFilterName ); + aTypeName = SfxFilter::GetTypeFromStorage( xStorage, pFilter && pFilter->IsAllowedAsTemplate(), &aTmpFilterName ); } catch( const WrappedTargetException& aWrap ) { |