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.