There _is_ no config file entry, unless I'm missing something you can point me
to. There was only a global bool flag in pcbnew.h that held the config value, I
was not able to find any place that was recorded to disk nor do I see any
evidence of it in my config files. Are you sure there is one?

On Wed, May 11, 2016 at 10:54:57AM +0200, Tomasz Wlostowski wrote:
> On 10.05.2016 23:41, Chris Pavlina wrote:
> > In 6790 I removed the user option to "Dump zone geometry to files when
> > filling"; Tom indicated that this was no longer necessary and could be 
> > removed.
> > 
> > I left the *code*, however. I took the global flag out of pcbnew.h and 
> > replaced
> > it with a static const bool in the file where the actual dumping happens, to
> > allow this to be switched back on in the future if absolutely necessary.
> > 
> > Particularly @ Tom: should the code be removed as well? How sure are we that
> > we'll never need it again? If "mostly sure", I'd advocate for removing it; 
> > in
> > the unlikely case that that changes we can drag it back out of the rev 
> > history.
> > If "kinda sure", maybe it can be left in.
> > 
> > What do you think?
> > 
> 
> I would drop the option from the dialog, but leave the code and the
> config file entry (so that it can be enabled by editing the config file
> only). Zone filling algorithm still has some rare corner case bugs and
> being able to see the intermediate steps of zone filling is a big aid in
> fixing them.
> 
> Cheers,
> Tom
> 

_______________________________________________
Mailing list: https://launchpad.net/~kicad-developers
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp

Reply via email to