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

Chr. Rossmanith ChrRossmanith at gmx.de
Mon Feb 27 06:04:24 PST 2012


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 method IIRC, only the string classes have 
> it. Please, check the types. :)
>
> Thanks,
> Ivan
Sorry, I thought, if the compiler doesn't complain everything is fine - 
but the compiler didn't see HtmlFmtFlt.cxx and had no chance to complain.

Thank you for reviewing,
Christina


More information about the LibreOffice mailing list