Date: prev next · Thread: first prev next last
2010 Archives by date, by thread · List index



Subject: [PATCH] [Janitor] remove 'precedence' warnings
Date: Thu, 30 Sep 2010 01:41:01 -0500
From: Norbert Thiebaud <norbert@lunodia.com>
To: libreoffice@list.freedesktop.org

This remove a couple of compile warning like
clone/writer/sw/source/ui/index/cnttab.cxx:3808: warning: suggest parentheses around && within ||

---
 sw/source/ui/index/cnttab.cxx |   10 +++++-----
 1 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/sw/source/ui/index/cnttab.cxx b/sw/source/ui/index/cnttab.cxx
index 5d9a390..7a1626e 100644
--- a/sw/source/ui/index/cnttab.cxx
+++ b/sw/source/ui/index/cnttab.cxx
@@ -1827,7 +1827,7 @@ void      SwTOXEdit::KeyInput( const KeyEvent& rKEvt )
     const Selection& rSel = GetSelection();
     sal_uInt16 nTextLen = GetText().Len();
     if(rSel.A() == rSel.B() &&
-        !rSel.A() || rSel.A() == nTextLen )
+       (!rSel.A() || rSel.A() == nTextLen ))
     {
         sal_Bool bCall = sal_False;
         KeyCode aCode = rKEvt.GetKeyCode();
@@ -3804,8 +3804,8 @@ IMPL_LINK(SwTokenWindow, EditResize, Edit*, pEdit)
 IMPL_LINK(SwTokenWindow, NextItemHdl, SwTOXEdit*,  pEdit)
 {
     sal_uInt16 nPos = (sal_uInt16)aControlList.GetPos(pEdit);
-    if(nPos && !pEdit->IsNextControl() ||
-        nPos < aControlList.Count() - 1 && pEdit->IsNextControl())
+    if((nPos && !pEdit->IsNextControl()) ||
+       (nPos < aControlList.Count() - 1 && pEdit->IsNextControl()))
     {
         aControlList.Seek(nPos);
         Control* pNextPrev = pEdit->IsNextControl() ? aControlList.Next() : aControlList.Prev();
@@ -3840,8 +3840,8 @@ IMPL_LINK(SwTokenWindow, TbxFocusHdl, SwTOXEdit*, pEdit)
 IMPL_LINK(SwTokenWindow, NextItemBtnHdl, SwTOXButton*, pBtn )
 {
     sal_uInt16 nPos = (sal_uInt16)aControlList.GetPos(pBtn);
-    if(nPos && !pBtn->IsNextControl() ||
-        nPos < aControlList.Count() - 1 && pBtn->IsNextControl())
+    if((nPos && !pBtn->IsNextControl()) ||
+       (nPos < aControlList.Count() - 1 && pBtn->IsNextControl()))
     {
         aControlList.Seek(nPos);
         sal_Bool bNext = pBtn->IsNextControl();
-- 

Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.