Hi,

On 02/11/2017 15:42, Jason Merrill wrote:

This is a good suggestion. I have attached the revised patch. Thanks.
OK, thanks!
Thanks Jason.

I was about to volunteer committing the patch but then noticed that the testcase includes quite a lot, eg, <inttypes.h> too, which we never include in the whole C++ testsuite. Can we have something simpler? Also, we don't need to include the whole <stdio.h> and <string.h> for a couple of declarations, we can simply provide by hand the declarations of sprintf and strcmp at the beginning of the file (plenty of examples in the testsuite). Mukesh, can you please work on that? Also, please watch out trailing blank lines.

Thanks,
Paolo.

Reply via email to