Re: [Libreoffice] [PATCH] [Janitor]: removed 'precedence' warnings.

Hi Norbert,

Could you please provide the issued warnings that you fixed? It seems
the code after the patch isn't equivalent to the code before.

--
Cedric

On Thu, 2010-09-30 at 01:43 -0500, nthiebaud@gmail.com wrote:
> ---
>  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();

-- 
Cédric Bosdonnat
LibreOffice hacker
http://documentfoundation.org
OOo Eclipse Integration developer
http://cedric.bosdonnat.free.fr