[Libreoffice] Same expression on both sides of '&&' on editeng/source/accessibility/AccessibleStaticTextBase.cxx
erack at redhat.com
Wed Feb 1 07:17:51 PST 2012
On Tuesday, 2012-01-31 13:36:41 -0800, julien2412 wrote:
> Should it be just replaced by this :
> if( aTextSegment.SegmentStart != -1 &&
> aTextSegment.SegmentEnd != -1 )
> or is it not so obvious ?
If only SegmentEnd was -1 and SegmentStart was not, then the state of
that segement would had been dubious and adding an offset to both
wouldn't make sense I think. So yes, this seems to be the solution.
Can't be worse than it was where it ignored SegmentEnd and looked like
a copy&paste error.
LibreOffice Calc developer. Number formatter stricken i18n transpositionizer.
GnuPG key 0x293C05FD : 997A 4C60 CE41 0149 0DB3 9E96 2F1A D073 293C 05FD
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Size: 198 bytes
Desc: not available
More information about the LibreOffice