[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 

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

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

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

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

Best Regards,

More information about the LibreOffice mailing list