This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

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


Hello Kai,

a couple of typo nits:

* Kai Tietz wrote on Thu, Mar 13, 2008 at 10:28:16AM CET:
> --- gcc.orig/gcc/doc/extend.texi
> +++ gcc/gcc/doc/extend.texi
> @@ -2204,13 +2204,22 @@ for consistency with the @code{printf} s
[...]
> +@code{__scanf__}, @code{__strftime__} or @code{__strfmon__}.)  On
> +MinGW targets there is also @code{ms_printf}, @code{ms_scanf}, and
> +@code{ms_strftime} present.

On MinGW targets, @code{...}..., and @code{...} are also present.

> --- gcc.orig/gcc/doc/tm.texi
> +++ gcc/gcc/doc/tm.texi
> @@ -10319,6 +10319,18 @@ If defined, this macro is the number of 
>  @code{TARGET_FORMAT_TYPES}.
>  @end defmac
>  
> +@defmac TARGET_OVERRIDES_FORMAT_ATTRIBUTES
> +If defined, this macro is the name of a global variable containg

s/containg/containing/

> +target-specific format overrides for the @option{-Wformat} option. The
> +default is to have no target-specific format overrides. If defined,
> +@code{TARGET_FORMAT_TYPES} must be defined too.
> +@end defmac

> --- /dev/null
> +++ gcc/gcc/testsuite/gcc.dg/format/ms_c99-printf-2.c
> @@ -0,0 +1,33 @@
[...]
> +  printf ("%qd", ll); /* { dg-warning "unknown|format" "%q length is unsupported" } */
> +  printf ("%Ld", ll); /* { dg-warning "unknown|format" "%L length is unsupported" } */
> +  printf ("%Zd", z); /* { dg-warning "unknown|format" "%Z lengthis unsupported" } */

s/lengthis/ length is/

Cheers,
Ralf


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]