[REVIEWED][PUSHED 3-5] fix for fdo#46220, infinite loop when exporting hidden column to html
kohei.yoshida at gmail.com
Fri Mar 23 07:20:24 PDT 2012
On Fri, 2012-03-23 at 04:40 +0100, Markus Mohrhard wrote:
>  fixes the infinite loop in html export when we try to skip a
> hidden column. We need to increase nCol in this case to prevent that
> we will always test the same column.
> This fix is safe and simple and I think we should include it into 3-5.
>  http://cgit.freedesktop.org/libreoffice/core/commit/?id=c4744121f12fde51de2d92d2442d590d93725ffc
Yup, I agree. Pushed to 3-5 with my sign-off.
BTW, I would probably make that loop into a for-loop instead of a
while-loop, since nCol value is not updated in each iteration, and there
is clearly a simple initialization step and the value gets incremented
at the end of each iteration. But, that's besides the issue, I guess
and some might call it just a personal opinion. ;-)
Kohei Yoshida, LibreOffice hacker, Calc
More information about the LibreOffice