[Libreoffice] [PATCH] Replace SvULongs with vector and code clean up part 1
Maciej Rumianowski
maciej.rumianowski at gmail.com
Sun Aug 7 08:04:49 PDT 2011
Hi Eike,
Dnia 2011-08-07, nie o godzinie 11:17 +0200, Eike Rathke pisze:
> You're right, the code is "suboptimal" ... in fact IsRemoved_Impl()
> and
> IsAdded_Impl() could also return the index of the element found so it
> wouldn't be needed to be searched again. Want to take care of it?
Yes :) My proposition of new declaration
> bool IsAdded_Impl( size_t nKey, size_t* nFoundKey );
I am attaching corrected patch, without this change and without size_t.
I have casted variables to size_t, in next patch I will try to change short to appropriate type and in other places try to use size_t.
Best Regards,
Maciej
More information about the LibreOffice
mailing list