kicad-developers team mailing list archive

[Dick Hollenbeck <dick@xxxxxxxxxxx>]

> Hi all, I'm just wanting to make sure that some warnings that I'm seeing
> are not intentional before I submit a patch to fix them.


The use of virtual, as is, is intentional.



> Clang's -Woverloaded-virtual warning [*] fires many times when building,
> primarily of the form:

Yes, just turn that off, the code is written the way we intended.


There is currently not enough protection against somebody changing the base, then you end
up with TWO virtuals.  This is why the derived ones are not using it.

I don't find that compiler option useful, at all.

Dick