Good morning, Wayne.
After sleeping on it, I think I can take another crack at this idea
that'll fit better with the points you've raised.
A couple things to clear up, just for the sake of same-pageness:
That being said, I still don't know exactly what
your are parsing and formatting. Is it the full symbol library file
format definition or just LIB_PART definition? If it's just LIB_PART
(which I suspect it is), then there is no need to break out all of the
symbol library file parser code. Only loadPart(), savePart(), and its
dependencies need to be broken out.
Correct. Just the LIB_PART parts, not the whole schematic. It is only
the LIB_PART parse/format code that I had split out into the static
classes, not the code related to reading schematics.
You could make them static in the SCH_LEGACY_PLUGIN_CACHE as well and
access them through the SCH_LEGACY_PLUGIN as static.
Agreed. After chewing on the ideas a while, I think this makes plenty
of sense for a second revision of my idea.
I'll have a new patch for review sometime soon.
Cheers,
-Brian
_______________________________________________
Mailing list: https://launchpad.net/~kicad-developers
Post to : kicad-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kicad-developers
More help : https://help.launchpad.net/ListHelp
