On Fri, 2012-03-02 at 22:13 -0500, Kohei Yoshida wrote:
> > [1] fixes a crash when you copy a chart in the document. The problem
> > is that you should not create a uno::Sequence with new because the
> > uno::Sequence copy c'tor is creating a flat copy. This will later
> > result in a double delete.
> >
> > I think the patch is quite save and fixes a crash and therefore should
> > be included into at least 3-5 and if still possible in 3-5-1.
> 
> > [1] 
> > http://cgit.freedesktop.org/libreoffice/core/commit/?id=8f2d3c47ad40039a842fa09d98137155dcfdfe9e
> 
> The change looks safe.  Pushed to the 3-5 branch with my sign-off.

        Ack from me too, trivial, safe fix fixing a crasher, fine for me on
-3-5-1, one more review needed.

        ATB,

                Michael.

-- 
[email protected]  <><, Pseudo Engineer, itinerant idiot

_______________________________________________
LibreOffice mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to