[Libreoffice] [REVIEW] Patch for "Same expression on both sides of '&&'" in basegfx module

Thorsten Behrens thb at documentfoundation.org
Fri Jan 6 08:46:27 PST 2012


Caolan McNamara wrote:
> > I think so, but I'm not sure either. Anyone with a second opinion?
> 
> Change seems reasonable to me FWIW, presumably Thorsten knows best what
> basegfx wants to do here.
> 
Trying to excercise the code properly via unit test currently, but
other stuff keeps distracting - so feel free to commit, it certainly
looks right, and I'll have a proper test rigged up in a bit to
validate.

Cheers,

-- Thorsten
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20120106/62ef72af/attachment.pgp>


More information about the LibreOffice mailing list