kicad-developers team mailing list archive

[Wayne Stambaugh <stambaughw@xxxxxxxxx>]

On 2/28/2017 4:37 AM, Maciej Sumiński wrote:
> Hi Wayne,
> 
> Would you have a look at the attached patches? I do not want to
> introduce changes to the schematic plugins code without your approval.
> 
> The first one removes multiple chunks of code that regenerate properties
> basing on the PART_LIB field values. Instead, a PROPERTIES object is
> maintained, so it is always ready whenever needed.

This patch is fine.  I already had this on my list of updates after I
complete the symbol library table.  Feel free to commit it as it cleans
up the property creation code.

> 
> Regarding the second patch: I am looking for a way to store temporary
> changes to libraries. Libraries are linked to files, and now the binding
> is even stronger, as files are reloaded upon a change [1].
> 
> It means that once I save only a part of modifications to a library, my
> buffer will be reloaded and the remaining unsaved changes will be gone.
> Do you think it is reasonable to disable cache updating when buffering
> is enabled?

I not sure I understand your question but the cache reloading doesn't
make sense when copying libraries and creating the schematic cache
library.  Otherwise, you get a library file write every new symbol you
add to the library.  On a schematic with a 100 different symbols, this
would be 100 file write operations just to save a library cache file.
That is why I added buffering to the legacy I/O plugin.  I know it's an
ugly hack but it's a stop gap measure until the new file format is in
place which will eliminate the need for library caching.  I suppose we
could change the cache reloading but that could have it's own set of
issues.  Technically the cache should only be loaded when the cache
hasn't been created, the library file name changes, or the file time
stamp has changed since the last file load.  If the file is being
reloaded other than these cases, then there is a bug in the cache
loading logic.

Cheers,

Wayne

> 
> Regards,
> Orson
> 
> 1. https://git.launchpad.net/kicad/tree/eeschema/sch_legacy_plugin.cpp#n3359
> 
> 
> 
> _______________________________________________
> 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
>