[Libreoffice] [PATCH] Replace SvULongs with vector and code clean up part 1

Maciej Rumianowski maciej.rumianowski at gmail.com
Sun Aug 7 12:49:16 PDT 2011

Dnia 2011-08-07, nie o godzinie 21:31 +0200, Eike Rathke pisze:
> I suggest
> ::std::vector<sal_uInt32>::iterator GetAdded_Impl( size_t nKey );
> with logically the content of IsAdded_Impl() but using ::std::find()
> instead of the awkward loop and returning the position's iterator that
> can be used in RemoveFormat(), plus
> bool IsAdded_Impl( size_t nKey )
> {
>     return GetAdded_Impl( nKey) != aAddList.end();
> }
> for all other places that need only a bool to remain the same.


> > in next patch I will try to change short to appropriate type and in other places try to use size_t.
> Take care not to touch too much, some of the short type seems to evolve
> from the list box using that type for position of elements. I think it
> doesn't pay to be overly aggressive there.

So I have tried to do this, but it is like Domino.

More information about the LibreOffice mailing list