Simon, Simon Richter had a patch with all or most of the overrides warnings fixed. I asked him to rebase it so I could get it merged because it was out of sync. That was a while ago. I haven't heard anything from him since then unless I missed it. I actually like the override since it specifically makes the intention of the developer known. As long as we don't abuse virtual in base classes, I would like to see this fixed now that we are officially a c+11 project rather than just disable the warning. I understand all the build warnings are annoying but we should start fixing them unless we have a good reason not to. This would be a big step forward in that regard.
Cheers, Wayne On 9/21/2016 4:01 PM, Simon Wells wrote: > --- > CMakeLists.txt | 5 +++++ > 1 file changed, 5 insertions(+) > > > > _______________________________________________ > Mailing list: https://launchpad.net/~kicad-developers > Post to : [email protected] > Unsubscribe : https://launchpad.net/~kicad-developers > More help : https://help.launchpad.net/ListHelp > _______________________________________________ Mailing list: https://launchpad.net/~kicad-developers Post to : [email protected] Unsubscribe : https://launchpad.net/~kicad-developers More help : https://help.launchpad.net/ListHelp
