[Libreoffice] [REVIEW]Fix for annoying bug fdo#35669

Cedric Bosdonnat cbosdonnat at suse.com
Sat Jan 28 00:25:49 PST 2012


Hi Michael,

On Thu, 2012-01-26 at 19:35 +0100, Michael Stahl wrote:
> >   sal_uInt16 nMaxDstId = *aIds.end();
> 
> oops, now it occurs to me that dereferencing end() is a bad idea in any
> case, you probably want to use rbegin() instead :)

I can only agree with you... I don't really see the subtleties there, if
you have some explanation, I'm all for it.

--
Cedric



More information about the LibreOffice mailing list