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

Muthu Subramanian K sumuthu at suse.com
Mon Feb 27 05:57:39 PST 2012


@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, aTextHtml and aData from your patch are 'Sequence's, Sequence
>> has not the isEmpty method IIRC, only the string classes have it.
> 
> fixed that now
> http://cgit.freedesktop.org/libreoffice/core/commit/?id=c581d33dec47a4c411a034482876543f6357d1e8
> 
> 
> Ivan
> _______________________________________________
> LibreOffice mailing list
> LibreOffice at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/libreoffice
> 



More information about the LibreOffice mailing list