Re: [Kicad-developers] [PATCH] Remove assert in pcbnew on osx with a debug build

2016-09-19 Thread Maciej Sumiński
Ok, I see. I have just removed the wxPaintDC object for all platforms, as anyway GAL worked fine without it. If there are any drawing related problems, let me know and we will look for another solution. Regards, Orson On 09/19/2016 02:25 PM, Simon Wells wrote: > Hi Orson, > > i get one of the

Re: [Kicad-developers] [PATCH] Remove assert in pcbnew on osx with a debug build

2016-09-19 Thread jp charras
Le 19/09/2016 à 13:53, Maciej Sumiński a écrit : > Hi Simon, > > Can you give more details about the assert? Perhaps it is a different > problem and should be solved in another way. > > wxWidgets documentation[1] says it is necessary to create a wxPaintDC > object, even if it is not used. IIRC

Re: [Kicad-developers] [PATCH] Remove assert in pcbnew on osx with a debug build

2016-09-19 Thread Maciej Sumiński
Hi Simon, Can you give more details about the assert? Perhaps it is a different problem and should be solved in another way. wxWidgets documentation[1] says it is necessary to create a wxPaintDC object, even if it is not used. IIRC GAL worked on every platform even when it has not created

Re: [Kicad-developers] [PATCH] Remove assert in pcbnew on osx with a debug build

2016-09-18 Thread Simon Wells
nickoe has commented on irc whether this is required on any platform, unfortunately i do not have any other platform for testing on, However the wxwidgets documention states that wxPaintDC should only be used in "native onpaint" handlers, what native means in this instance i am unsure, however on

[Kicad-developers] [PATCH] Remove assert in pcbnew on osx with a debug build

2016-09-17 Thread Simon Wells
--- common/draw_panel_gal.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/common/draw_panel_gal.cpp b/common/draw_panel_gal.cpp index fd48d83..2616490 100644 --- a/common/draw_panel_gal.cpp +++ b/common/draw_panel_gal.cpp @@ -156,7 +156,9 @@ void EDA_DRAW_PANEL_GAL::SetFocus() void

[Kicad-developers] [PATCH] Remove assert in pcbnew on osx with a debug build

2016-09-17 Thread Simon Wells
--- common/draw_panel_gal.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/common/draw_panel_gal.cpp b/common/draw_panel_gal.cpp index fd48d83..2616490 100644 --- a/common/draw_panel_gal.cpp +++ b/common/draw_panel_gal.cpp @@ -156,7 +156,9 @@ void EDA_DRAW_PANEL_GAL::SetFocus() void