On Mon, 2012-02-27 at 00:53 -0600, Norbert Thiebaud wrote:
> > Fixed invalid conversion from SvUShorts to std::vector... *sigh*
> > Please, pick to the appropriate branches.
>
> I pushed it on the 3.5 branch.
With one more review (preferably from a writer maintainer) we're good
for 3.5.1 I think. RC2 is final though Ivan - so we're trying to reduce
the number of patches between RC1 and final of our point releases.
Having said that I'm not hyper-happy with not initialising the vector's
contents - don't we really want:
std::vector<sal_uInt16> aWish( aTablCols.Count() + 1, 0 ),
aMins( aTabCols.Count() + 1, 0 );
The previous Max() of the count with 255 looks somewhat odd /
disturbing - no idea what horrors that might hide ;-) sadly that sort of
thing has been there since the beginning.
Anyhow - thanks for fixing the bug !
Regards,
Michael.
--
michael.me...@suse.com <><, Pseudo Engineer, itinerant idiot
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice
