kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #28651
Re: [PATCH] Refactor LAYER_ID to be the one and only layer definition
Jon,
I misunderstood your original intent. I don't think cluttering the
board layer enums with all of the virtual layer and schematic layer
enums is a good idea. It just seems like overkill to me. I thought you
were going to create a separate enum for virtual board layers. You
could always start the virtual board layer enums from the last board
layer enum if you need unique enums. I would also prefer the schematic
layer enums be separate from the board layer enums for clarity. Anyone
else have any thoughts on this?
Cheers,
Wayne
On 3/12/2017 11:24 PM, Jon Evans wrote:
> Hi all,
>
> Per the other thread, this patch unifies the layer definitions between
> Pcbnew, GerbView, and Eeschema. It removes the need for ITEM_GAL_LAYER
> and some other macros, and it will simplify the implementation of
> cross-application color themes and using GAL in multiple applications.
>
> Note that this patch introduces some temporary weirdness in a few
> places, such as in COLORS_DESIGN_SETTINGS (there is now a single array
> for color storage, but it's still referred to by two sets of
> getters/setters). This is because I wanted to keep this refactor as
> simple as possible, as I plan to follow it up with an overhaul of color
> settings when I share my color themes work. I didn't want to do work
> that I would soon end up getting rid of anyway.
>
> Best,
> 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
>
Follow ups
References