kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #28077
Re: [PATCH] Change from EDA_COLOR_T to COLOR4D and arbitrary color support
-
To:
kicad-developers@xxxxxxxxxxxxxxxxxxx
-
From:
jp charras <jp.charras@xxxxxxxxxx>
-
Date:
Wed, 22 Feb 2017 19:15:57 +0100
-
In-reply-to:
<CAHg9EZpDZD=AsmQN_ACoXNXJ0oS1R3dvwtoh4pmpbSYQE7cOnQ@mail.gmail.com>
-
User-agent:
Mozilla/5.0 (Windows NT 6.1; rv:45.0) Gecko/20100101 Thunderbird/45.7.1
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