[PATCH] Replace getLength() with isEmpty() where appropriate

2012-02-27 Thread Chr. Rossmanith
getLength() with isEmpty() where appropriate --- vcl/aqua/source/app/salinst.cxx |6 +++--- vcl/aqua/source/app/salsys.cxx |4 ++-- vcl/aqua/source/dtrans/HtmlFmtFlt.cxx |4 ++-- vcl/aqua/source/dtrans/PictToBmpFlt.cxx |4 ++-- vcl/source/gdi/pdfwriter_impl.cxx

Re: [Pushed] [PATCH] Replace getLength() with isEmpty() where appropriate

2012-02-27 Thread Muthu Subramanian K
Pushed. Thank you! On 02/27/2012 01:52 PM, Chr. Rossmanith wrote: Hi, some more getLength() - isEmpty() replacements. Would be nice if someone could review the patch. Christina ___ LibreOffice mailing list

Re: [PATCH] Replace getLength() with isEmpty() where appropriate

2012-02-27 Thread Ivan Timofeev
Hi Christina, On 27.02.2012 12:22, Chr. Rossmanith wrote: some more getLength() - isEmpty() replacements. Would be nice if someone could review the patch. at least, aTextHtml and aData from your patch are 'Sequence's, Sequence has not the isEmpty method IIRC, only the string classes have it.

Re: [PATCH] Replace getLength() with isEmpty() where appropriate

2012-02-27 Thread Ivan Timofeev
On 27.02.2012 16:59, Ivan Timofeev wrote: Hi Christina, On 27.02.2012 12:22, Chr. Rossmanith wrote: some more getLength() - isEmpty() replacements. Would be nice if someone could review the patch. at least, aTextHtml and aData from your patch are 'Sequence's, Sequence has not the isEmpty

Re: [PATCH] Replace getLength() with isEmpty() where appropriate

2012-02-27 Thread Muthu Subramanian K
@Ivan: Thanks! I missed that one. On 02/27/2012 06:45 PM, Ivan Timofeev wrote: On 27.02.2012 16:59, Ivan Timofeev wrote: Hi Christina, On 27.02.2012 12:22, Chr. Rossmanith wrote: some more getLength() - isEmpty() replacements. Would be nice if someone could review the patch. at least,

Re: [PATCH] Replace getLength() with isEmpty() where appropriate

2012-02-27 Thread Chr. Rossmanith
Am 27.02.2012 13:59, schrieb Ivan Timofeev: Hi Christina, On 27.02.2012 12:22, Chr. Rossmanith wrote: some more getLength() - isEmpty() replacements. Would be nice if someone could review the patch. at least, aTextHtml and aData from your patch are 'Sequence's, Sequence has not the isEmpty