[Libreoffice] [PATCH] some refactoring in xmlcelli.cxx

Kohei Yoshida kohei.yoshida at gmail.com
Sat Aug 27 11:09:35 PDT 2011


Hi Markus,

On Sat, Aug 27, 2011 at 9:24 AM, Markus Mohrhard
<markus.mohrhard at googlemail.com> wrote:

> It would be nice if someone could give it a quick review before I push it.

Looks good to me.  The second hunk was a bit hard to grok, but I came
to the same conclusion as you did.

> Especially the long if statement is a bit tricky but I think that only the
> first line was needed.

Yup.  I agree.  Suffice it to say that that if statement is one of the
stinkiest I've seen in this code base.

Kohei


More information about the LibreOffice mailing list