[Libreoffice] [PATCH][PUSHED] Replace List for vector<> in sc

Petr Mladek pmladek at suse.cz
Wed Jun 8 06:30:30 PDT 2011


Rafael Dominguez píše v Út 07. 06. 2011 v 19:05 -0430:
> Made a new patch, with the fixes now im using size_t to handle size,
> also ensuring we dont add a null object.

I have pushed it as 
http://cgit.freedesktop.org/libreoffice/calc/commit/?id=31a268eb01d9d808a35ffe3161acbec7d4134519

Well, I have pushed the older patch and just replaced sal_uInt16 with
size_t.

IMHO, the check for existing pObj is not needed. It seems that the
method "Add" is always called with valid objects... If we start
supporting NULL parameter, do we really want to return 0 or the current
vector size?

Well, this whole code need some clean up and get rid of the sal_uInt16
limit...


Anyway, this was the last patch from this set => adding PUSHED into the
subject.

Thanks a lot for the contribution! It helps a lot to make the code
cleaner.


Best Regards,
Petr



More information about the LibreOffice mailing list