diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2014-01-23 18:43:53 +0100 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2014-01-23 18:43:53 +0100 |
commit | e3133ae2371b6e2d4b47daa6a3487ea4f2753315 (patch) | |
tree | f9a000ee6542fb470733bfed28199092a36c26c5 /svl | |
parent | 14f4e2fa22a0a2576808bab08edaf2e12f761cd8 (diff) |
Let C++ inline functions return bool instead of sal_Bool
...to improve diagnosing misuses of boolean expressions in client code (cf.
compilerplugins/clang/implicitboolconversion.cxx). This change should be
transparent to client code.
Missing overloads of insert() for bool have been added to OStringBuffer and
OUStringBuffer (which required dropping one !VALID_CONVERSION check that would
now pick that overload, but would be flagged by
compilerplugins/clang/pointertobool.cxx).
Change-Id: I2d64cd923b8f47bfaa31e753def6515c29a3f8c9
Diffstat (limited to 'svl')
-rw-r--r-- | svl/source/items/poolio.cxx | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/svl/source/items/poolio.cxx b/svl/source/items/poolio.cxx index 0eb0add9ae54..b96c2d6d5245 100644 --- a/svl/source/items/poolio.cxx +++ b/svl/source/items/poolio.cxx @@ -546,7 +546,7 @@ SvStream &SfxItemPool::Load(SvStream &rStream) } // Einzel-Header - int bOwnPool = sal_True; + bool bOwnPool = true; OUString aExternName; { // Header-Record suchen @@ -730,7 +730,7 @@ SvStream &SfxItemPool::Load1_Impl(SvStream &rStream) rStream >> pImp->nMajorVer >> pImp->nMinorVer; } sal_uInt32 nAttribSize(0); - int bOwnPool = sal_True; + bool bOwnPool = true; OUString aExternName; if ( pImp->nMajorVer > 1 || pImp->nMinorVer >= 2 ) rStream >> pImp->nLoadingVersion; |