← Back to team overview

kicad-developers team mailing list archive

Re: bug in numeric_evaluator?

 

+Michael

This looks like "clever" code so I can't tell if this is a bug or not
without running through a bunch of test cases which I'm not going to do
right now.
Do you see any issues with the operation of the code, Mark, or just the
warning?

-Jon

On Mon, Mar 12, 2018 at 9:28 PM, Mark Roszko <mark.roszko@xxxxxxxxx> wrote:

> Not sure where to even begin looking at libeval since it looks scary but
> MSVC complains:
>
> D:\kicad-source-mirror\common\libeval\numeric_evaluator.cpp(216): warning
> C4805: '==': unsafe mix of type 'char' and type 'bool' in operation
>
> which is:
>
>       auto ch = clToken.input[clToken.pos];
>       do {
>          if (ch == isDecSep(ch) && sepCount) break;
>
>
> ch is a char
>
> isDecSep returns bool
>
> char == bool?
>
>
>
> --
> Mark
>
> _______________________________________________
> 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