← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Change from EDA_COLOR_T to COLOR4D and arbitrary color support

 

Le 22/02/2017 à 19:05, Diogo Condeço a écrit :
> Hi Jon,
> 
> Your patch 0001 on this thread introduced a bug which makes kicad unable to build...
> 
> a52250a91e24733ce798ad8baa4597032d49d11e this was the commit.
> 
> kicad/common/draw_panel.cpp:757:15: error: no matching constructor for initialization of 'wxPen'
>         wxPen pen( GetParent()->GetGridColor(), h );
> 
> 
> This is on a macos machine with clang.
> 
> Thanks,
> 
> Diogo
> 
> 

On Windows, an other issue with latest commits:
In file included from E:/kicad-launchpad/testing_git/common/tool/tool_manager.cpp:44:0:
E:/kicad-launchpad/testing_git/include/tool/coroutine.h:228:37: error: macro "Yield" passed 1
arguments, but takes just 0
     void Yield( ReturnType& aRetVal )

I am thinking Yield(...) collides with an other definition somewhere, perhaps in wxWidgets (which
already defines Yield)

Renaming Yield to KiYield fixes it (in tool_manager.cpp and coroutine.h)

-- 
Jean-Pierre CHARRAS


References