[Libreoffice] [REVIEWED] Re: [REVIEW] Regression from 3.3
Jan Holesovsky
kendy at suse.cz
Fri Jun 17 07:09:30 PDT 2011
Hi,
Kohei Yoshida píše v Út 14. 06. 2011 v 14:06 -0400:
> I would like to cherry-pick the following commit on master
>
> http://cgit.freedesktop.org/libreoffice/calc/commit/?id=478533a86b29341e00128fb6eda8237dfb9cf3cf
>
> to the -3-4 branch (and -3-4-1 if it makes sense). There is no bug
> report associated with this (yet), but this accidentally changes the
> behavior of ScMatrix pre- and post-refactoring. When GetString() is
> called on an empty element, the old code would return an empty string,
> but the new code sets an error instead. I made this change, and this
> change was unintentional. GetString() should still return an empty
> string when called on an empty element.
>
> Review & sign-off appreciated.
Looks good to me, and behaving according to the explanation :-) - pushed
to libreoffice-3-4. Two more reviews to get it to -3-4-1 needed.
Thank you,
Kendy
More information about the LibreOffice
mailing list