Re: [Libreoffice] [PATCH] SvUShorts -> std::vector

Thanks for the advice, I will keep this in mind.


Best,
Daniel


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@gmx.de
> <mailto:d.dimarco@gmx.de>> wrote:
>
>     Hi,
>
>     I replaced some usages of SvUShorts with std::vectors.
>     License is LGPLv3+/MPL
>
>
>     Cheers,
>     Daniel
>
>     _______________________________________________
>     LibreOffice mailing list
>     LibreOffice@lists.freedesktop.org
>     <mailto:LibreOffice@lists.freedesktop.org>
>     http://lists.freedesktop.org/mailman/listinfo/libreoffice