On Fri, 24 Apr 2009, Ranier Vilela wrote:
Hi,
> macroy.c(1797) : warning C4065: switch statement contains 'default' but no
> 'case' labels
> macroy.c:
> line 1792: switch (yytype)
> line 1793: {
> line 1794:
> line 1795: default:
> line 1796: break;
> line 1797: }
This code is generated by bison. To pacify the above warning
I will have to add some dummy code. I do not know if it's good idea here.
I'll look at it but if the cost of introducing dummy destructors for
terminal symbol will cause RT overhead then I'll do not add them.
> cl.exe -nologo -I. -I../../../../include -Gs -TP -W4 -wd4127 -Wall -O2xtyb1
> -Gy -GS- -GR- -EHs- -DNDEBUG -DHB_FM_STATISTICS_OFF -MT -c
> ../../complex.c -Focomplex.obj
> complex.c
> ../../complex.c(510) : warning C4806: '==' : unsafe operation: no value of
> type 'bool' promoted to type 'int' can equal the given constant
> complex.c:
> 510: if( !pToken->len == 3 || pToken->value[ 2 ] != '0' )
> 511: iYear = -1;
> -> Maybe if( pToken->len == 3 || pToken->value[2] != '0')
> // without the ! operator
No. This is the 1-st real bug in our source code shown by extended MSVC
messages and the warning message is very precise. It's quite common mistake
when I have to switch between C and .prg code - the ! operator has
different precedence. The correct line is:
if( pToken->len != 3 || pToken->value[2] != '0' )
Thanks for the info. I'll commit fix ASAP.
best regards,
Przemek
_______________________________________________
Harbour mailing list
Harbour@harbour-project.org
http://lists.harbour-project.org/mailman/listinfo/harbour
