Why are you doing this: switch( m_style & 0xFF )
Why is that necessary when the default case should handle any invalid values? Especially because the enum MARKER_STYLE has no assigned values nor really is a bitmasking type of enum. On Thu, Sep 3, 2015 at 5:37 PM, Brian Sidebotham <[email protected]> wrote: > To match functionality of the Legacy canvas I propose we extend the > GAL ORIGIN_VIEWITEM class to allow us to add an axis origin marker > when using the Footprint Editor. I'm sure even while in the stable > feature-lock this is okay to commit, but an ack will be good before I > go ahead. > > Also, Orson, thanks for your information about this, can you also ack > that you're happy with the class extension? Otherwise feel free to > contact me to implement this in another way if you'd rather... > > Best Regards, > > Brian. > > _______________________________________________ > Mailing list: https://launchpad.net/~kicad-developers > Post to : [email protected] > Unsubscribe : https://launchpad.net/~kicad-developers > More help : https://help.launchpad.net/ListHelp > -- Mark _______________________________________________ Mailing list: https://launchpad.net/~kicad-developers Post to : [email protected] Unsubscribe : https://launchpad.net/~kicad-developers More help : https://help.launchpad.net/ListHelp
