You can commit that, but I can't quite tell why there are two different functions ReFillRender() and SyncRenderStates(); they do mostly the same thing so it seems best to just collapse them into one to avoid bugs in the future. The attached patch removes SyncRenderStates -- please let me know if this makes sense to you also.
Thanks, -Jon On Fri, Feb 23, 2018 at 10:05 AM, jp charras <jp.char...@wanadoo.fr> wrote: > Le 23/02/2018 à 15:57, Jon Evans a écrit : > > If you add a call to ReFillRender() at the top of > > PCB_LAYER_WIDGET::SyncRenderStates() > instead of my > > change, does it fix it? > > > > Yes, just adding the call fixes this issue. > Can I commit this fix? > > > > On Fri, Feb 23, 2018 at 9:54 AM, jp charras <jp.char...@wanadoo.fr > <mailto:jp.char...@wanadoo.fr>> > > wrote: > > > > Le 23/02/2018 à 14:24, Jon Evans a écrit : > > > Hi JP, good catch, the attached fixes it for me > > > > > > -Jon > > > > > > On Fri, Feb 23, 2018 at 7:18 AM, jp charras <jp.char...@wanadoo.fr > > <mailto:jp.char...@wanadoo.fr> <mailto:jp.char...@wanadoo.fr > <mailto:jp.char...@wanadoo.fr>>> > > > wrote: > > > > > > Le 23/02/2018 à 01:04, Jon Evans a écrit : > > > > Here are two more quick patches addressing some of the > easier changes. > > > > I have some other bugs to look at that seem more urgent, so > I'm not going to work on the right-click > > > > menu or settings persistence. > > > > > > > > Thanks, > > > > Jon > > > > > > > > > > Hi Jon, > > > > > > There is a (minor but annoying) issue about patch: > > > > > > "Only show microvia and blind/buried via settings if they are > enabled" > > > > > > When I load a board that have microvia and blind/buried vias > allowed, the layer manager does not > > > show the microvia and blind/buried vias menuitems. > > > > > > They are displayed only after opening and closing by OK button > the Design Rules menu. > > > > > > (Tested on W7/32bits) > > > > > > -- > > > Jean-Pierre CHARRAS > > > > With this patch, Pcbnew crashes when starting, but not always. > > I am thinking m_Layers->ReFillRender(); is called too early, when > initializations are not yet > > finished > > > > In fact it is called before m_Layers is initialized. > > > > The fix is easy, but when loading a file, syncRenderStates(); is > called > > (files_io.cpp line 592) > > > > Perhaps the root issue is SyncRenderStates() is missing some > initializations. > > > > > > > > -- > > Jean-Pierre CHARRAS > > > > > > > -- > Jean-Pierre CHARRAS >
Description: Binary data
_______________________________________________ Mailing list: https://launchpad.net/~kicad-developers Post to : email@example.com Unsubscribe : https://launchpad.net/~kicad-developers More help : https://help.launchpad.net/ListHelp