Hi Markus,

On Sat, Aug 27, 2011 at 9:24 AM, Markus Mohrhard
<[email protected]> 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
_______________________________________________
LibreOffice mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to