← Back to team overview

kicad-developers team mailing list archive

Re: Fix: Wrong GetScreen() in pl_editor

 

Bernhard,

Your change works, bitmap2component does not crash on exit anymore. BZR 4945 in Bernhard branch.

Jean-Paul
AC9GH


On Jun 15, 2014, at 2:31 PM, Bernhard Stegmaier <stegmaier@xxxxxxxxxxxxx> wrote:

> Hi,
> 
> while fixing some clang compile warnings I added an obviously missing const to GetScreen() in pl_editor_frame.h (part of http://bazaar.launchpad.net/~stegmaier/kicad/kicad-osx/revision/4936) to fix an "overloaded virtual" warning.
> 
> This forced the compiler to now really take the function defined in pl_editor_frame.h… and pl_editor didn’t start up any longer.
> Changing GetScreen() like it is done in other applications to just use the function of the base class, it is fine again (and the same behavior as without the fixed const):
>  http://bazaar.launchpad.net/~stegmaier/kicad/kicad-osx/revision/4944
> 
> Although it doesn’t fix any visible bug right now, I guess it should be fixed for clarity.
> 
> 
> Regards,
> Bernhard
> _______________________________________________
> 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



References