Re: [PATCH, GCC] Deal with singular sentences in builtin-sprintf-warn-1.c regex

2016-09-26 Thread Jeff Law
On 09/26/2016 03:55 AM, Thomas Preudhomme wrote: How about this reworked patch? Best regards, Thomas On 23/09/16 17:17, Martin Sebor wrote: On 09/23/2016 09:42 AM, Thomas Preudhomme wrote: Sorry, forgot the patch. Please find it attached. Best regards, Thomas On 23/09/16 16:40, Thomas Pre

Re: [PATCH, GCC] Deal with singular sentences in builtin-sprintf-warn-1.c regex

2016-09-26 Thread Thomas Preudhomme
How about this reworked patch? Best regards, Thomas On 23/09/16 17:17, Martin Sebor wrote: On 09/23/2016 09:42 AM, Thomas Preudhomme wrote: Sorry, forgot the patch. Please find it attached. Best regards, Thomas On 23/09/16 16:40, Thomas Preudhomme wrote: Hi, New builtin-sprintf-warn-1.c

Re: [PATCH, GCC] Deal with singular sentences in builtin-sprintf-warn-1.c regex

2016-09-23 Thread Thomas Preudhomme
Hi Martin, On 23/09/16 17:17, Martin Sebor wrote: On 09/23/2016 09:42 AM, Thomas Preudhomme wrote: Sorry, forgot the patch. Please find it attached. Best regards, Thomas On 23/09/16 16:40, Thomas Preudhomme wrote: Hi, New builtin-sprintf-warn-1.c testcase contains a few regex of the form "

Re: [PATCH, GCC] Deal with singular sentences in builtin-sprintf-warn-1.c regex

2016-09-23 Thread Martin Sebor
On 09/23/2016 09:42 AM, Thomas Preudhomme wrote: Sorry, forgot the patch. Please find it attached. Best regards, Thomas On 23/09/16 16:40, Thomas Preudhomme wrote: Hi, New builtin-sprintf-warn-1.c testcase contains a few regex of the form "\[0-9\]+ bytes" or ". bytes". This does not account

Re: [PATCH, GCC] Deal with singular sentences in builtin-sprintf-warn-1.c regex

2016-09-23 Thread Thomas Preudhomme
Sorry, forgot the patch. Please find it attached. Best regards, Thomas On 23/09/16 16:40, Thomas Preudhomme wrote: Hi, New builtin-sprintf-warn-1.c testcase contains a few regex of the form "\[0-9\]+ bytes" or ". bytes". This does not account for the case where the number of byte is 0 in whic

[PATCH, GCC] Deal with singular sentences in builtin-sprintf-warn-1.c regex

2016-09-23 Thread Thomas Preudhomme
Hi, New builtin-sprintf-warn-1.c testcase contains a few regex of the form "\[0-9\]+ bytes" or ". bytes". This does not account for the case where the number of byte is 0 in which case byte would be in the singular form. This caused a FAIL on arm-none-eabi targets. This patch makes the s optio