Re: Ping - old patch from April - mingw support for I32/I64 MS printf formatters to c-format.c

[Kai Tietz <Kai dot Tietz at onevision dot com>]

"Danny Smith" <dansmister@gmail.com> wrote on 13.03.2008 21:40:40:

> >  >
> >  > I see 44 format.exp failures in testsuite  with your patch
> >  I corrected the test-cases so far.
> 
> Well I suppose one way to correct the test case is just to remove some
> tests, but your revised patch doesn't really address the problems with
> strftime.
> Attached is log of "make -k check-gcc RUNTESTFLAGS=format.exp" using
> prior patch.  Do you not see these errors?

Thank you for the log file. I correct the tested cases so far. The major 
problem is that by your suggestion 'I64' length specifier isn't treated as 
equivalent to the gnu 'll' length sepecifier. This is the reason for the 
failure of a lot of warnings. I let the test-case ms_warnll-1.c as it is 
for showing the issue. I64 and co are declared as STD_EXT, which is in my 
opinion not correct. They should be set as STD_C890 IMHO.

I attached the revised patch.

Cheers,
 i.A. Kai Tietz



|  (\_/)  This is Bunny. Copy and paste Bunny
| (='.'=) into your signature to help him gain
| (")_(") world domination.

Attachment: mingw-msformat.txt
Description: Text document