src/bison/src/location.c:192: poor choice of type for local variable ?

[David Binderman]

Hello there,

I just ran static analysis tool "cppcheck" over the source code
of the latest version of bison.

It said

[src/bison/src/location.c:192]: (warning) Storing getc() return value in char 
variable and then comparing with EOF.

Source code is

    char c = getc (caret_info.source);
    if (c != EOF)

The if test will always fail. Suggest change type of local variable c to int.

Regards

David Binderman