On Sat, Jan 14, 2012 at 04:08:46AM -0800, julien2412 wrote: > Hello, > > cppcheck reported this : > core/sw/source/ui/shells/frmsh.cxx > 516 duplicateExpression style Same expression on both sides of '&&' > > After a quick reading, i thought about this page but perhaps it's less > straightforward than I think > > diff --git a/sw/source/ui/shells/frmsh.cxx b/sw/source/ui/shells/frmsh.cxx > index 718839c..7a6db74 100644 > --- a/sw/source/ui/shells/frmsh.cxx > +++ b/sw/source/ui/shells/frmsh.cxx > @@ -513,7 +513,7 @@ void SwFrameShell::Execute(SfxRequest &rReq) > if(pArgs && SFX_ITEM_SET == > pArgs->GetItemState(GetPool().GetWhich(nSlot), sal_False, &pItem)) > { > aMgr.SetAttrSet( *pArgs ); > - if(SID_ATTR_ULSPACE == nSlot && SID_ATTR_ULSPACE == nSlot) > + if(SID_ATTR_ULSPACE == nSlot && SID_ATTR_LRSPACE == nSlot) > bCopyToFmt = sal_True; > } > } > > Any idea ?
Yes, you changed the condition from "always true" to "always false" .-) I think it is not necessary at all. It is hard to tell what was the intent, because the code has not changed since the original CVS import. D. _______________________________________________ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice