[Libreoffice] [PUSHED][PATCH] Fix for fdo43460 Part XIV getLength() to isEmpty()
Ivan Timofeev
timofeev.i.s at gmail.com
Fri Dec 23 10:13:38 PST 2011
Hi Gustavo,
23.12.2011 21:21, Gustavo Pacheco пишет:
> Hi!
>
> Partial fix for Easy Hack FDO43460 (part XIV) for review. Mentoring by
> Olivier Hallot (thanks for the tips Olivier!).
>
> Part XVI
> Modules drawinglayer, dtrans and editeng
Looks nice for me, but
- if (aLocale.Language.getLength() ||
aLocale.Country.getLength())
+ if (!(aLocale.Language.isEmpty() || aLocale.Country.isEmpty()))
is wrong - (!A || !B) => !(A && B), but since I personally don't like
negators outside a compound condition, I changed it to the first variant.
Pushed to master:
http://cgit.freedesktop.org/libreoffice/core/commit/?id=b575f4b1a2a2217282cddc995951b350936b47b1
Thank you Gustavo, thank you Olivier!
Ivan
More information about the LibreOffice
mailing list