kicad-developers team mailing list archive

[Chris Pavlina <pavlina.chris@xxxxxxxxx>]

I feel like emphasizing here that our attempt at saving file size doesn't
actually do anything since we've changed defaults in our library standard
anyway. :\

On Thu, Mar 10, 2016 at 11:01:32PM +0000, Jon Neal wrote:
> Oh, I should add that to prevent breaking backwards compatibility the
> parser could just continue interpreting a missing size and thickness value
> with the current defaults hard coded there.
> 
> On Thu, Mar 10, 2016 at 5:59 PM Jon Neal <reportingsjr@xxxxxxxxx> wrote:
> 
> > 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
> >

> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help   : https://help.launchpad.net/ListHelp