kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #23763
Questionably useful code
Hi,
I was looking in to helping the library team by changing the default kicad
text size to 50 mils rather than the current 60 mils.
Well, I discovered that the s-expr formatter and parser omits certain text
settings if they are the default. I assume this is to make kicad files
smaller which yay, but it means that we basically can't change the default
text size without breaking backwards compatibility. Huge, resounding BOOOOO.
What I would like to request is that we remove the bit of code that omits
text size and thickness if they are default. This increases file size by
about 40ish chars/bytes. I think it is reasonable to still omit bold,
italic, justification, and whether the text is hidden or not.
On a fairly dense board (thanks Andrew Zonenberg) there are about 1.3k
fp_text's so this would theoretically add ~50kB to a several MB file.
Except it doesn't add nearly that much, because the kicad library doesn't
use the current default size anyways. So for a large portion of text this
code is useless anyways.
Offending code is in common/eda_text.cpp:407
Thoughts?
Jon
Follow ups