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

Rafael Dominguez venccsralph at gmail.com
Tue Sep 20 08:44:33 PDT 2011


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> wrote:

> Hi,
>
> I replaced some usages of SvUShorts with std::vectors.
> License is LGPLv3+/MPL
>
>
> Cheers,
> Daniel
>
> _______________________________________________
> LibreOffice mailing list
> LibreOffice at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/libreoffice
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20110920/ee767f67/attachment-0001.htm>


More information about the LibreOffice mailing list