[Libreoffice] [PATCH] SvUShorts -> std::vector
Daniel Di Marco
d.dimarco at gmx.de
Tue Sep 20 16:32:00 PDT 2011
Thanks for the advice, I will keep this in mind.
On 09/20/2011 03:44 PM, Rafael Dominguez wrote:
> Just a little input about patch, nothing serious but its worth saying,
> when checking if a container has elements, dont use the size() call
> since its O(n) and not a constant call on some stl implementations, its
> preferred to use empty().
> On Mon, Sep 19, 2011 at 7:51 PM, Daniel Di Marco <d.dimarco at gmx.de
> <mailto:d.dimarco at gmx.de>> wrote:
> I replaced some usages of SvUShorts with std::vectors.
> License is LGPLv3+/MPL
> LibreOffice mailing list
> LibreOffice at lists.freedesktop.org
> <mailto:LibreOffice at lists.freedesktop.org>
More information about the LibreOffice