As far as I can tell, this patch causes the following compilation error in the generated flex code, when using Werror=sign-compare

 In function ‘yy_get_next_buffer’:
error: comparison between signed and unsigned integer expressions [-Werror=sign-compare] if ((int) (yyg->yy_n_chars + number_to_move) > YY_CURRENT_BUFFER_LVALUE->yy_buf_size) {

On Wed, 24 Feb 2016 17:45:14 -0800 Manoj Srivastava <sriva...@debian.org> wrote:
>
> tags 770161 + pending
> thanks
> Hi,
>
> The following change has been committed for this bug by
> Manoj Srivastava <sriva...@debian.org> on the branch
> master at Wed, 24 Feb 2016 17:44:01 -0800.
>
> The fix will be in the next upload.
> =========================================================================
> [master]: Cleanup generated files based on changes to flex.skl.
>
> The variable yy_n_chars had been of type yy_size_t which is incorrect
> given its use in read(). While it might be adviseable to look at
> defining a yy_ssize_t, there might be some issues doing this and so, for
> now, at least, we'll punt back to int.
>
> Bug fix: "Type of yy_n_chars changed to yy_size_t by authors cause bug
> in YY_INPUT macro where result argument is tested if &lt; 0; thanks to
> gcc&#39;s -Werror=type-limits", thanks to Michal Fita
> (Closes: #770161).
>
> Signed-off-by: Manoj Srivastava <sriva...@debian.org>
> =========================================================================
>
>
>

Reply via email to