fontmanager

Olivier Hallot <olivier.hallot -AT- documentfoundation.org>
Thu, 12 Jan 2012 14:17:37 -0200

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Thank you Christina for the patch.

It look like your patch is ok, with respect to the logic inversion. A
build should indicate if the method isEmpty() is valid for the right
object (e.g. it is not valid for OUStringBuffer or Sequence).

Do you mind to
1) update bug fdo43460 with your (pushed) patch?

2) complete module vcl/ with these replacements? I will mark it as yours
in my control file. Attached an exerpt of the files and lines to look at.

Regards

Olivier


Em 12-01-2012 12:29, Chr. Rossmanith escreveu:

Hi,

while replacing ByteString with OString I came across some getLength()
calls which might be replaced with isEmpty(). Hope Olivier won't be
annoyed  :-)

Some reviewing would be nice to be sure the logic didn't get inverted
accidently.I have git push access - so reviewing would be sufficient.

Christina


_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice


- -- 
Olivier Hallot
Founder, Board of Directors Member - The Document Foundation
LibreOffice translation leader for Brazilian Portuguese
+55-21-8822-8812
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.11 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iQEcBAEBAgAGBQJPDwehAAoJEJp3R7nH3vLxl3cH/0qtyGFDI44WaSWr5XVKgU7n
oGzqcz/DBAuaDRr7NZnHp2rerAXNur1r6M1gVL+yyO/DsLxwnDdUAQsUdwcZspzR
zckK5D8l9q6oD/+o16Ismr5/r+E9Fn5Jz/MH+ER+nqslZe5k138eJZH2L5KrwR7i
J1Uq5ZIZHa+Q9+aWCBeRVpb8RU+VnBx2nVBUQ98ovQ/ShDCE3SNGOwNSQAQtz8si
LzXd4MfTmFZJSk4DwlTXXOAoXcYn6rwRdxr2DsyOa/faOeBwWrBVLrKfGN4Bl1rn
MYEDLvQqZQ7T3FHik4aOHVUmvfYvJKaP1X+oAcHgM5WJpHhYmURf1wAMY8QgrI4=
=tufN
-----END PGP SIGNATURE-----

/data/lo/core/vcl/generic/app/gensys.cxx:159:9: note: [EmptyLength plugin] replace "getLength() != 
0" with "!isEmpty()"
/data/lo/core/vcl/generic/fontmanager/fontcache.cxx:179:21: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/generic/fontmanager/fontcache.cxx:236:17: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/generic/fontmanager/fontconfig.cxx:1003:5: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/generic/fontmanager/fontconfig.cxx:767:5: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/generic/fontmanager/fontconfig.cxx:778:5: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/generic/fontmanager/fontconfig.cxx:856:13: note: [EmptyLength plugin] replace 
"getLength() > 0" with "!isEmpty()"
/data/lo/core/vcl/generic/fontmanager/fontconfig.cxx:921:5: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/generic/fontmanager/fontconfig.cxx:988:5: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/generic/fontmanager/fontconfig.cxx:992:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/generic/fontmanager/fontconfig.cxx:999:5: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/generic/fontmanager/fontmanager.cxx:1697:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/generic/fontmanager/fontmanager.cxx:1702:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/generic/fontmanager/fontmanager.cxx:1898:5: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/generic/fontmanager/fontmanager.cxx:1949:13: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/generic/fontmanager/fontmanager.cxx:2165:5: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/generic/fontmanager/fontmanager.cxx:2174:13: note: [EmptyLength plugin] replace 
"getLength() == 0" with "isEmpty()"
/data/lo/core/vcl/generic/fontmanager/fontmanager.cxx:3281:17: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/generic/fontmanager/fontmanager.cxx:3447:5: note: [EmptyLength plugin] replace 
"getLength() == 0" with "isEmpty()"
/data/lo/core/vcl/generic/fontmanager/fontmanager.cxx:3497:13: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/generic/fontmanager/fontmanager.cxx:3573:17: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/generic/fontmanager/fontmanager.cxx:792:9: note: [EmptyLength plugin] replace 
"getLength() == 0" with "isEmpty()"
/data/lo/core/vcl/generic/fontmanager/fontmanager.cxx:806:5: note: [EmptyLength plugin] replace 
"getLength() == 0" with "isEmpty()"
/data/lo/core/vcl/generic/fontmanager/helper.cxx:160:9: note: [EmptyLength plugin] replace 
"getLength() == 0" with "isEmpty()"
/data/lo/core/vcl/generic/fontmanager/helper.cxx:204:5: note: [EmptyLength plugin] replace 
"getLength() == 0" with "isEmpty()"
/data/lo/core/vcl/generic/fontmanager/helper.cxx:211:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/generic/fontmanager/helper.cxx:228:9: note: [EmptyLength plugin] replace 
"getLength() == 0" with "isEmpty()"
/data/lo/core/vcl/generic/fontmanager/helper.cxx:230:13: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/generic/fontmanager/helper.cxx:237:13: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/generic/glyphs/gcach_ftyp.cxx:603:5: note: [EmptyLength plugin] replace 
"getLength() == 0" with "isEmpty()"
/data/lo/core/vcl/generic/print/common_gfx.cxx:1196:5: note: [EmptyLength plugin] replace 
"getLength() == 0" with "isEmpty()"
/data/lo/core/vcl/generic/print/common_gfx.cxx:1231:5: note: [EmptyLength plugin] replace 
"getLength() == 0" with "isEmpty()"
/data/lo/core/vcl/generic/print/printerjob.cxx:292:5: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/generic/print/printerjob.cxx:406:5: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/generic/print/printerjob.cxx:476:69: note: [EmptyLength plugin] replace 
"getLength() > 0" with "!isEmpty()"
/data/lo/core/vcl/inc/generic/gendata.hxx:63:9: note: [EmptyLength plugin] replace "getLength() == 
0" with "isEmpty()"
/data/lo/core/vcl/inc/image.h:90:73: note: [EmptyLength plugin] replace "getLength() != 0" with 
"!isEmpty()"
/data/lo/core/vcl/source/app/settings.cxx:1779:5: note: [EmptyLength plugin] replace "getLength() 
== 0" with "isEmpty()"
/data/lo/core/vcl/source/app/settings.cxx:1881:5: note: [EmptyLength plugin] replace "getLength() 
== 0" with "isEmpty()"
/data/lo/core/vcl/source/app/settings.cxx:1892:5: note: [EmptyLength plugin] replace "getLength() 
== 0" with "isEmpty()"
/data/lo/core/vcl/source/app/settings.cxx:738:5: note: [EmptyLength plugin] replace "getLength() > 
0" with "!isEmpty()"
/data/lo/core/vcl/source/app/unohelp.cxx:112:21: note: [EmptyLength plugin] replace "getLength() != 
0" with "!isEmpty()"
/data/lo/core/vcl/source/control/edit.cxx:980:5: note: [EmptyLength plugin] replace "getLength() != 
0" with "!isEmpty()"
/data/lo/core/vcl/source/control/tabctrl.cxx:1382:13: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/control/tabctrl.cxx:2033:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/control/tabctrl.cxx:632:9: note: [EmptyLength plugin] replace "getLength() 
== 0" with "isEmpty()"
/data/lo/core/vcl/source/control/tabctrl.cxx:637:9: note: [EmptyLength plugin] replace "getLength() 
== 0" with "isEmpty()"
/data/lo/core/vcl/source/control/throbber.cxx:218:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/configsettings.cxx:165:17: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/gfxlink.cxx:403:9: note: [EmptyLength plugin] replace "getLength() != 
0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/image.cxx:799:5: note: [EmptyLength plugin] replace "getLength() != 0" 
with "!isEmpty()"
/data/lo/core/vcl/source/gdi/image.cxx:850:13: note: [EmptyLength plugin] replace "getLength() != 
0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/impgraph.cxx:1044:17: note: [EmptyLength plugin] replace "getLength() 
!= 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/impgraph.cxx:1256:13: note: [EmptyLength plugin] replace "getLength() 
!= 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/impimage.cxx:114:5: note: [EmptyLength plugin] replace "getLength() != 
0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/impimage.cxx:121:5: note: [EmptyLength plugin] replace "getLength() != 
0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/impimage.cxx:98:9: note: [EmptyLength plugin] replace "getLength() != 
0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/impimagetree.cxx:213:13: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/impimagetree.cxx:219:17: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/impimagetree.cxx:254:5: note: [EmptyLength plugin] replace 
"getLength() == 0" with "isEmpty()"
/data/lo/core/vcl/source/gdi/pdfwriter_impl.cxx:10829:9: note: [EmptyLength plugin] replace 
"getLength() > 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/pdfwriter_impl.cxx:10951:5: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/pdfwriter_impl.cxx:1101:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/pdfwriter_impl.cxx:11702:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/pdfwriter_impl.cxx:11733:9: note: [EmptyLength plugin] replace 
"getLength() == 0" with "isEmpty()"
/data/lo/core/vcl/source/gdi/pdfwriter_impl.cxx:2876:9: note: [EmptyLength plugin] replace 
"getLength() > 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/pdfwriter_impl.cxx:2887:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/pdfwriter_impl.cxx:2893:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/pdfwriter_impl.cxx:2903:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/pdfwriter_impl.cxx:2910:5: note: [EmptyLength plugin] replace 
"getLength() > 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/pdfwriter_impl.cxx:2914:9: note: [EmptyLength plugin] replace 
"getLength() > 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/pdfwriter_impl.cxx:4626:13: note: [EmptyLength plugin] replace 
"getLength() > 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/pdfwriter_impl.cxx:4687:25: note: [EmptyLength plugin] replace 
"getLength() > 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/pdfwriter_impl.cxx:4712:21: note: [EmptyLength plugin] replace 
"getLength() > 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/pdfwriter_impl.cxx:4722:21: note: [EmptyLength plugin] replace 
"getLength() > 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/pdfwriter_impl.cxx:5407:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/pdfwriter_impl.cxx:5462:25: note: [EmptyLength plugin] replace 
"getLength() == 0" with "isEmpty()"
/data/lo/core/vcl/source/gdi/pdfwriter_impl.cxx:5525:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/pdfwriter_impl.cxx:5531:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/pdfwriter_impl.cxx:5645:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/pdfwriter_impl.cxx:5647:13: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/pdfwriter_impl.cxx:5670:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/pdfwriter_impl.cxx:5955:5: note: [EmptyLength plugin] replace 
"getLength() > 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/pdfwriter_impl.cxx:5959:9: note: [EmptyLength plugin] replace 
"getLength() > 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/pdfwriter_impl.cxx:780:5: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/pdfwriter_impl.cxx:795:5: note: [EmptyLength plugin] replace 
"getLength() == 0" with "isEmpty()"
/data/lo/core/vcl/source/gdi/pdfwriter_impl2.cxx:1217:5: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/pdfwriter_impl2.cxx:1224:97: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/print3.cxx:1386:13: note: [EmptyLength plugin] replace "getLength() > 
0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/print3.cxx:1709:47: note: [EmptyLength plugin] replace "getLength() != 
0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/print3.cxx:1711:5: note: [EmptyLength plugin] replace "getLength() != 
0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/print3.cxx:1722:5: note: [EmptyLength plugin] replace "getLength() != 
0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/print3.cxx:1739:5: note: [EmptyLength plugin] replace "getLength() != 
0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/print3.cxx:1754:5: note: [EmptyLength plugin] replace "getLength() != 
0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/print3.cxx:1782:5: note: [EmptyLength plugin] replace "getLength() > 
0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/print3.cxx:1796:5: note: [EmptyLength plugin] replace "getLength() > 
0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/print3.cxx:1813:5: note: [EmptyLength plugin] replace "getLength() > 
0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/print3.cxx:1872:5: note: [EmptyLength plugin] replace "getLength() > 
0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/print3.cxx:1896:5: note: [EmptyLength plugin] replace "getLength() > 
0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/print3.cxx:360:9: note: [EmptyLength plugin] replace "getLength() != 
0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/print3.cxx:385:17: note: [EmptyLength plugin] replace "getLength() == 
0" with "isEmpty()"
/data/lo/core/vcl/source/gdi/print3.cxx:443:17: note: [EmptyLength plugin] replace "getLength() == 
0" with "isEmpty()"
/data/lo/core/vcl/source/gdi/print3.cxx:527:9: note: [EmptyLength plugin] replace "getLength() != 
0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/rendergraphic.cxx:118:35: note: [EmptyLength plugin] replace 
"getLength() == 0" with "isEmpty()"
/data/lo/core/vcl/source/gdi/rendergraphic.cxx:69:5: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/gdi/rendergraphicrasterizer.cxx:246:13: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/helper/strhelper.cxx:177:5: note: [EmptyLength plugin] replace 
"getLength() == 0" with "isEmpty()"
/data/lo/core/vcl/source/helper/strhelper.cxx:386:5: note: [EmptyLength plugin] replace 
"getLength() == 0" with "isEmpty()"
/data/lo/core/vcl/source/window/menu.cxx:2030:9: note: [EmptyLength plugin] replace "getLength() != 
0" with "!isEmpty()"
/data/lo/core/vcl/source/window/menu.cxx:2039:17: note: [EmptyLength plugin] replace "getLength() 
!= 0" with "!isEmpty()"
/data/lo/core/vcl/source/window/menu.cxx:2089:9: note: [EmptyLength plugin] replace "getLength() != 
0" with "!isEmpty()"
/data/lo/core/vcl/source/window/menu.cxx:848:13: note: [EmptyLength plugin] replace "getLength() == 
0" with "isEmpty()"
/data/lo/core/vcl/source/window/msgbox.cxx:226:5: note: [EmptyLength plugin] replace "getLength() 
!= 0" with "!isEmpty()"
/data/lo/core/vcl/source/window/printdlg.cxx:1184:83: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/window/printdlg.cxx:1188:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/window/printdlg.cxx:1227:18: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/window/printdlg.cxx:1236:77: note: [EmptyLength plugin] replace 
"getLength() == 0" with "isEmpty()"
/data/lo/core/vcl/source/window/printdlg.cxx:1257:14: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/window/printdlg.cxx:1259:69: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/window/printdlg.cxx:1370:17: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/window/printdlg.cxx:1437:17: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/window/printdlg.cxx:163:5: note: [EmptyLength plugin] replace "getLength() 
!= 0" with "!isEmpty()"
/data/lo/core/vcl/source/window/printdlg.cxx:1724:17: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/window/printdlg.cxx:2263:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/window/printdlg.cxx:269:5: note: [EmptyLength plugin] replace "getLength() 
!= 0" with "!isEmpty()"
/data/lo/core/vcl/source/window/printdlg.cxx:991:5: note: [EmptyLength plugin] replace "getLength() 
!= 0" with "!isEmpty()"
/data/lo/core/vcl/source/window/status.cxx:1441:9: note: [EmptyLength plugin] replace "getLength() 
!= 0" with "!isEmpty()"
/data/lo/core/vcl/source/window/status.cxx:1448:17: note: [EmptyLength plugin] replace "getLength() 
!= 0" with "!isEmpty()"
/data/lo/core/vcl/source/window/status.cxx:1504:9: note: [EmptyLength plugin] replace "getLength() 
!= 0" with "!isEmpty()"
/data/lo/core/vcl/source/window/status.cxx:890:13: note: [EmptyLength plugin] replace "getLength() 
!= 0" with "!isEmpty()"
/data/lo/core/vcl/source/window/status.cxx:898:26: note: [EmptyLength plugin] replace "getLength() 
!= 0" with "!isEmpty()"
/data/lo/core/vcl/source/window/toolbox.cxx:4581:9: note: [EmptyLength plugin] replace "getLength() 
!= 0" with "!isEmpty()"
/data/lo/core/vcl/source/window/toolbox.cxx:4588:17: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/window/toolbox.cxx:4657:13: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/window/toolbox.cxx:4665:26: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/window/toolbox2.cxx:1788:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/source/window/window.cxx:8326:25: note: [EmptyLength plugin] replace "getLength() 
!= 0" with "!isEmpty()"
/data/lo/core/vcl/source/window/window.cxx:8329:25: note: [EmptyLength plugin] replace "getLength() 
!= 0" with "!isEmpty()"
/data/lo/core/vcl/source/window/window2.cxx:1341:88: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/generic/app/saldata.cxx:459:5: note: [EmptyLength plugin] replace 
"getLength() == 0" with "isEmpty()"
/data/lo/core/vcl/unx/generic/app/saldisp.cxx:317:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/generic/app/sm.cxx:478:53: note: [EmptyLength plugin] replace "getLength() != 
0" with "!isEmpty()"
/data/lo/core/vcl/unx/generic/app/wmadaptor.cxx:1011:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/generic/app/wmadaptor.cxx:1016:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/generic/app/wmadaptor.cxx:908:9: note: [EmptyLength plugin] replace 
"getLength() == 0" with "isEmpty()"
/data/lo/core/vcl/unx/generic/dtrans/X11_selection.cxx:380:92: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/generic/dtrans/X11_selection.cxx:711:5: note: [EmptyLength plugin] replace 
"getLength() == 0" with "isEmpty()"
/data/lo/core/vcl/unx/generic/dtrans/config.cxx:98:17: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/generic/printer/cupsmgr.cxx:265:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/generic/printer/cupsmgr.cxx:608:9: note: [EmptyLength plugin] replace 
"getLength() > 0" with "!isEmpty()"
/data/lo/core/vcl/unx/generic/printer/cupsmgr.cxx:694:17: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/generic/printer/jobdata.cxx:57:5: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/generic/printer/ppdparser.cxx:1162:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/generic/printer/ppdparser.cxx:1164:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/generic/printer/ppdparser.cxx:142:9: note: [EmptyLength plugin] replace 
"getLength() == 0" with "isEmpty()"
/data/lo/core/vcl/unx/generic/printer/ppdparser.cxx:1514:5: note: [EmptyLength plugin] replace 
"getLength() == 0" with "isEmpty()"
/data/lo/core/vcl/unx/generic/printer/ppdparser.cxx:1524:5: note: [EmptyLength plugin] replace 
"getLength() == 0" with "isEmpty()"
/data/lo/core/vcl/unx/generic/printer/ppdparser.cxx:179:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/generic/printer/ppdparser.cxx:184:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/generic/printer/ppdparser.cxx:189:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/generic/printer/ppdparser.cxx:211:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/generic/printer/ppdparser.cxx:216:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/generic/printer/printerinfomanager.cxx:1126:87: note: [EmptyLength plugin] 
replace "getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/generic/printer/printerinfomanager.cxx:597:5: note: [EmptyLength plugin] 
replace "getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/generic/printer/printerinfomanager.cxx:730:13: note: [EmptyLength plugin] 
replace "getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/generic/printer/printerinfomanager.cxx:736:9: note: [EmptyLength plugin] 
replace "getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/generic/printer/printerinfomanager.cxx:769:9: note: [EmptyLength plugin] 
replace "getLength() == 0" with "isEmpty()"
/data/lo/core/vcl/unx/generic/printer/printerinfomanager.cxx:906:9: note: [EmptyLength plugin] 
replace "getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/generic/window/salframe.cxx:2160:65: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/generic/window/salframe.cxx:3103:5: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/generic/window/salframe.cxx:3128:5: note: [EmptyLength plugin] replace 
"getLength() > 0" with "!isEmpty()"
/data/lo/core/vcl/unx/generic/window/salframe.cxx:3185:41: note: [EmptyLength plugin] replace 
"getLength() > 0" with "!isEmpty()"
/data/lo/core/vcl/unx/gtk/a11y/atktext.cxx:115:13: note: [EmptyLength plugin] replace "getLength()

0" with "!isEmpty()"

/data/lo/core/vcl/unx/gtk/a11y/atktext.cxx:129:13: note: [EmptyLength plugin] replace "getLength()

0" with "!isEmpty()"

/data/lo/core/vcl/unx/gtk/a11y/atktext.cxx:83:5: note: [EmptyLength plugin] replace "getLength() > 
0" with "!isEmpty()"
/data/lo/core/vcl/unx/gtk/a11y/atktextattributes.cxx:547:5: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/gtk/a11y/atktextattributes.cxx:558:5: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/gtk/app/gtkdata.cxx:675:5: note: [EmptyLength plugin] replace "getLength() > 
0" with "!isEmpty()"
/data/lo/core/vcl/unx/gtk/fpicker/SalGtkFilePicker.cxx:1392:9: note: [EmptyLength plugin] replace 
"getLength() == 0" with "isEmpty()"
/data/lo/core/vcl/unx/gtk/fpicker/SalGtkFilePicker.cxx:1875:13: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/gtk/fpicker/SalGtkFilePicker.cxx:1877:17: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/gtk/fpicker/SalGtkFilePicker.cxx:1927:5: note: [EmptyLength plugin] replace 
"getLength() == 0" with "isEmpty()"
/data/lo/core/vcl/unx/gtk/fpicker/SalGtkFilePicker.cxx:1960:17: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/gtk/fpicker/SalGtkFilePicker.cxx:1999:5: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/gtk/fpicker/SalGtkFilePicker.cxx:2001:10: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/gtk/fpicker/SalGtkFilePicker.cxx:604:9: note: [EmptyLength plugin] replace 
"getLength() == 0" with "isEmpty()"
/data/lo/core/vcl/unx/gtk/fpicker/SalGtkFilePicker.cxx:871:33: note: [EmptyLength plugin] replace 
"getLength() == 0" with "isEmpty()"
/data/lo/core/vcl/unx/gtk/fpicker/SalGtkPicker.cxx:107:9: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/gtk/fpicker/SalGtkPicker.cxx:90:13: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/gtk/window/gtkframe.cxx:1947:89: note: [EmptyLength plugin] replace 
"getLength() != 0" with "!isEmpty()"
/data/lo/core/vcl/unx/gtk/window/gtkframe.cxx:4124:9: note: [EmptyLength plugin] replace 
"getLength() == 0" with "isEmpty()"

Context

[Libreoffice] [PATCH] Replace getLength() with isEmpty() in vcl/generic/fontmanager · Chr. Rossmanith
- Re: [Libreoffice] [PATCH] Replace getLength() with isEmpty() in vcl/generic/fontmanager · Olivier Hallot
  - Re: [Libreoffice] [PATCH] Replace getLength() with isEmpty() in vcl/generic/fontmanager · Chr. Rossmanith
    - Re: [Libreoffice] [PATCH] Replace getLength() with isEmpty() in vcl/generic/fontmanager · Olivier Hallot
- [Libreoffice] [PUSHED, partial][PATCH] Replace getLength() with isEmpty() in vcl/generic/fontmanager · Caolán McNamara

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.