Le 23/02/2018 à 16:12, Jon Evans a écrit :
> 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
> 

I committed you patch.

It makes sense to me, and this is the right way to fix this issue.
Sometimes code cleanup is needed.

Thanks.

> 
> On Fri, Feb 23, 2018 at 10:05 AM, jp charras <jp.char...@wanadoo.fr 
> <mailto: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> <mailto: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>>
>     <mailto: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
> 
> 


-- 
Jean-Pierre CHARRAS

_______________________________________________
Mailing list: https://launchpad.net/~kicad-developers
Post to     : kicad-developers@lists.launchpad.net
Unsubscribe : https://launchpad.net/~kicad-developers
More help   : https://help.launchpad.net/ListHelp

Reply via email to