Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upMinify XML output #414
Open
Minify XML output #414
Assignees
Labels
Comments
|
I didn't minify it yet as it makes debugging more difficult, but it makes sense to allow the developer to choose. I'll make this configurable. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I was testing xlnt output against the output from libxlsxwriter, when writing the same data the xlnt xlsx file was always larger. After doing some digging it appears that the internal xml files had whitespace enable (they are not minified).
It appears that I only had to change the follow line to minify the xml outputs:
xlnt/source/detail/serialization/xlsx_producer.cpp
Line 182 in f302601
This is the change I made on my fork: https://github.com/mattparks/xlnt/commit/6c32ca8745843aec77f58e7a904b8fca5a62fca5