Skip to content

C
core
Kaydet (Commit) a45c79da authored tarafından Julien Nabet's avatar Julien Nabet
Dosyalara gözat
Seçenekler

cppcheck: Clarify calculation precedence for '%' and '?' 

Change-Id: I59f9763651ba576a5d0dc970283d46a18f806024
üst 43627f2a
Hide whitespace changes
Inline Side-by-side
Showing with 2 additions and 2 deletions
sw/source/core/layout/pagechg.cxx
Dosyayı görüntüle @ a45c79da
...@@ -1287,7 +1287,7 @@ SwPageFrm *SwFrm::InsertPage( SwPageFrm *pPrevPage, sal_Bool bFtn ) ...@@ -1287,7 +1287,7 @@ SwPageFrm *SwFrm::InsertPage( SwPageFrm *pPrevPage, sal_Bool bFtn )
if ( rDesc.GetNumOffset() ) if ( rDesc.GetNumOffset() )
{ {
::boost::optional<sal_uInt16> oNumOffset = rDesc.GetNumOffset(); ::boost::optional<sal_uInt16> oNumOffset = rDesc.GetNumOffset();
bWishedOdd = (oNumOffset ? oNumOffset.get() : 0) % 2 ? true : false; bWishedOdd = ((oNumOffset ? oNumOffset.get() : 0) % 2) ? true : false;
//Die Gelegenheit nutzen wir um das Flag an der Root zu pflegen. //Die Gelegenheit nutzen wir um das Flag an der Root zu pflegen.
pRoot->SetVirtPageNum( sal_True ); pRoot->SetVirtPageNum( sal_True );
} }
...@@ -1568,7 +1568,7 @@ void SwRootFrm::AssertFlyPages() ...@@ -1568,7 +1568,7 @@ void SwRootFrm::AssertFlyPages()
{ {
//Die Seiten werden ausgehend von der letzten Seite konsequent //Die Seiten werden ausgehend von der letzten Seite konsequent
//nach den Regeln der PageDescs weitergefuehrt. //nach den Regeln der PageDescs weitergefuehrt.
sal_Bool bOdd = pPage->GetPhyPageNum() % 2 ? sal_True : sal_False; sal_Bool bOdd = (pPage->GetPhyPageNum() % 2) ? sal_True : sal_False;
SwPageDesc *pDesc = pPage->GetPageDesc(); SwPageDesc *pDesc = pPage->GetPageDesc();
SwFrm *pSibling = pPage->GetNext(); SwFrm *pSibling = pPage->GetNext();
for ( i = pPage->GetPhyPageNum(); i < nMaxPg; ++i ) for ( i = pPage->GetPhyPageNum(); i < nMaxPg; ++i )
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment