← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Simple teardrops implementation

 

This fix disables the menu entry in legacy mode.

diff --git a/pcbnew/pcbframe.cpp b/pcbnew/pcbframe.cpp
index 931335b..52f98fa 100644
--- a/pcbnew/pcbframe.cpp
+++ b/pcbnew/pcbframe.cpp
@@ -730,7 +730,8 @@ void PCB_EDIT_FRAME::enableGALSpecificMenus()
             ID_TUNE_SINGLE_TRACK_LEN_BUTT,
             ID_TUNE_DIFF_PAIR_LEN_BUTT,
             ID_TUNE_DIFF_PAIR_SKEW_BUTT,
-            ID_MENU_DIFF_PAIR_DIMENSIONS
+            ID_MENU_DIFF_PAIR_DIMENSIONS,
+            ID_TEARDROPS_WINDOW,
         };

         bool enbl = IsGalCanvasActive();

On Sun, May 10, 2015 at 4:24 AM, Nick Østergaard <oe.nick@xxxxxxxxx> wrote:

> I noticed that it does not segfault in the default canvas, but the legacy
> canvas. Is this patch only supposed to work with the GAL canvas?
> Den 10/05/2015 06.14 skrev "Михаил Карпенко" <veegboo@xxxxxxxxx>:
>
>> Hi, Nick.
>>
>>
>> Thanks for testing this out. I checked the patch against recent version
>> of
>>
>> KiCad which I cloned from guthub today and could not reproduce the
>> segfault. I
>>
>> used the coldfire demo you mentioned as well. Which version of KiCad did
>> you
>>
>> use for testing?
>>
>

References