kicad-developers team mailing list archive

["Marco Serantoni" <marco.serantoni@...>]

--- In kicad-devel@xxxxxxxxxxxxxxx, Dick Hollenbeck <dick@...> wrote:

> I would say find a way to fix this so we do not need the
> #ifndef __WXMAX__
> Maybe Wayne or Jean-Pierre and see a way to this.
> It may simply be that we don't need the Affiche_Status_Box() calls for 
> any OS at these specific locations. It seems like that would be true 
> if Affiche_Status_Box() is managing a different "window or widget" at 
> the operating system level.
> It may be that a call to this function AT THIS LOCATION is a relic that 
> was obsoleted after I re-wrote the Draw() function in the status window 
> a year or so ago.

On OSX the change is not affecting the correct behaviour of the statusbar, that still be refreshed by Mouse and other events. 
Better implementations are welcome.
Due my limited knowledge of the codebase i've proposed a conservative patchjust to point and fix the problem and avoid to broke some other platforms:As Hippocrates used to say: "First, do no harm." :)

Evidence of problems seems be:
Thu Apr 2 22:18:21 snowball.local pcbnew[23127] <Error>: CGContextRestoreGState: invalid context
Thu Apr 2 22:18:21 snowball.local pcbnew[23127] <Error>: CGContextRestoreGState: invalid context