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 /forms/source | |
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 'forms/source')
-rw-r--r-- | forms/source/component/Grid.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/forms/source/component/Grid.cxx b/forms/source/component/Grid.cxx index 1be7b6d48ac0..4275ef5a02fc 100644 --- a/forms/source/component/Grid.cxx +++ b/forms/source/component/Grid.cxx @@ -959,7 +959,7 @@ void OGridControlModel::read(const Reference<XObjectInputStream>& _rxInStream) t m_bEnable = _rxInStream->readBoolean(); if (nAnyMask & TABSTOP) { - m_aTabStop = makeBoolAny(_rxInStream->readBoolean()); + m_aTabStop = makeBoolAny(_rxInStream->readBoolean() != 0); } if (nVersion > 3) m_bNavigation = _rxInStream->readBoolean(); |