Bug 810

Summary: configlexer.c: comparison of integers of different signs
Product: NSD Reporter: ruga
Component: NSD CodeAssignee: NSD team <nsd-team>
Status: REOPENED ---    
Severity: enhancement CC: wouter
Priority: P5    
Version: 4.1.x   
Hardware: x86_64   
OS: other   

Description ruga 2016-08-05 09:07:23 CEST
claration-after-statement -c configlexer.c
<stdout>:2354:23: warning: comparison of integers of different signs: 'yy_size_t' (aka 'unsigned long') and 'int' [-Wsign-compare]
                if ( number_to_move == YY_MORE_ADJ )
                     ~~~~~~~~~~~~~~ ^  ~~~~~~~~~~~
Comment 1 Wouter Wijngaards 2016-08-05 09:10:30 CEST
Hi Ruga

This is caused by code created by flex.  I have seen other versions of this tool not give this warning, so it is possible to fix (or update the version?).  But otherwise, the warning is harmless here, I believe.

So, it is not something I can fix, the warning is created by the code created by flex.

Best regards, Wouter
Comment 2 ruga 2016-08-05 09:31:18 CEST
confirmed

It would be useful to add a note to doc/REQUIREMENTS and
a version check into configure.

thank you
Comment 3 Wouter Wijngaards 2016-08-05 09:31:56 CEST
Hi Ruga,

Do you know what the version is that worked for you (and the version that failed) ?

Best regards, Wouter
Comment 4 ruga 2016-08-07 18:22:11 CEST
>/usr/bin/flex --version
flex 2.5.35 Apple(flex-31)

/usr/local/bin/flex --version
flex 2.6.1

Both versions return the same problem.
This little bug is still roaming...
Comment 5 Wouter Wijngaards 2016-08-08 09:03:49 CEST
Hi Ruga,

flex 2.5.39 on Fedora works fine.  I think it may be the configuration of flex itself at install time (eg. signedness of particular types).

I don't know if there is anything I can do about it.

Best regards, Wouter