← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] compile broken on clang - overload conflict

 

Really!  Clang can't differentiate between

SCH_SHEET_LIST::GetSheet(const wxString, bool);

and

SCH_SHEET_LIST::GetSheet(int);

How can these two definitions be ambiguous?  Does using a reference to
the wxString instead of passing the entire string on the stack fix the
problem?  If so, I would prefer that fix.

On 6/7/2015 9:17 PM, Chris Pavlina wrote:
> 5720 broke the build on clang by making SCH_SHEET_LIST::GetSheet(int) a 
> const method, which changed the overload resolution order and resulted 
> in failure due to ambiguity. This patch ranames 
> SCH_SHEET_LIST::GetSheet(const wxString, bool) to ::GetSheetByPath to 
> resolve that conflict.
> 
> --
> Chris
> 
> 
> 
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help   : https://help.launchpad.net/ListHelp
> 


Follow ups

References