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


I'd like to draw your attention on my commit

ommit b090cbdf82e0827234caf5969124f6631311ef35
Author: Lionel Elie Mamane <lionel@mamane.lu>
Date:   Fri Aug 30 18:00:21 2013 +0200

    fdo#68663 don't blindly force categories when there are none

    Change-Id: If52ea471ee3ca10e2f69d2fcd7b3ae0d5772bfc3


If the comment is right (which is not always the case in the
LibreOffice codebase...), then I've just broken reading of categories
for these kinds of old "files with own data without a categories
element but with row descriptions".

This code was pretty much unconditionally always adding categories,
although I could see no trace of "row description" in the charts
generated by Report Builder; by force-adding categories, this was
leading to fdo#68663.

So, if anyone has a clue how to make that "force-add categories"
conditional to the presence of these mysterious "row descriptions",
I'd be happy to collaborate to add this (now properly conditional)
"add categories" back.

We would also need to revert the subsequent -Werror cleanup

commit ccf7b15c0a5776c6431fdcb0c0e2b0f3935ae3dc
Author: Julien Nabet <serval2412@yahoo.fr>
Date:   Fri Aug 30 21:02:26 2013 +0200

    -Werror=unused-but-set-variable second try

    Change-Id: Ia8a9f60831d089ec04127213bb9ded31aa754bf7


I also don't quite understand why the removed code was not passing
"&rLSequencesPerIndex" as 6th argument. Shouldn't it have?


-- 
Lionel

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.