Date: prev next · Thread: first prev next last
2011 Archives by date, by thread · List index


On Fri, 2011-07-01 at 10:18 +0200, Jan Holesovsky wrote:
Hi Cedric, all,

https://bugs.freedesktop.org/show_bug.cgi?id=37516

So this one was a particularly nasty one; the only thing I had was a
black box (in the form of MSO 2010) answering yes/no about the document
validity :-)  Long story short, it was SPRM SDxtCharSpace that had wrong
value.

The attached patch reverts the simplified computation of its value that
produces wrong results when nPitch is smaller than nPageCharSize.  Maybe
it could be simplified in another way, but I think the safest ATM is to
return to the algorithm that was there before (see commit
e5693d5e14f8a53070b86ea1682201f5f2149b82).

Review appreciated, targeting libreoffice-3-4 branch.

Thanks for your help and patch for that bug... I though I would become
mad ;) patch looks good to me... just pushed it.

-- 
Cédric Bosdonnat
LibreOffice hacker
http://documentfoundation.org
OOo Eclipse Integration developer
http://cedric.bosdonnat.free.fr


Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.