Re: [ros-dev] [ros-diffs] [jgardou] 54793: [NEWINFLIB] - use _snwprintf instead of swprintf (MSVC is not compliant with standard here)

[Thomas Faber]

Hey,

On 2011-12-31 15:33, jgardou(a)svn.reactos.org wrote:
>   Author: jgardou
>  Date: Sat Dec 31 14:33:35 2011
>  New Revision: 54793
>  
>  URL: http://svn.reactos.org/svn/reactos?rev=54793&view=rev
>  Log:
>  [NEWINFLIB]
>  - use _snwprintf instead of swprintf (MSVC is not compliant with standard here) 
since _snwprintf seems unavailable on Unix, perhaps
-D_CRT_NON_CONFORMING_SWPRINTFS
would be an appropriate solution? That should cause VS2010 (2008 seems to have
the version without a length) to use the non-standard no-length version just
like GCC.
That [1] gets rid of the warnings for me.

Regards (and a happy new year :p),
Tom

[1] http://pastebin.com/4XU2SCXR

[Jérôme Gardou]

Hey Thomas.

I thought _snwprintf was a standard function. Apparently I was wrong.

This is confusing. Is _CRT_NON_CONFORMING_SWPRINTFS a valid GCC define 
too, or does it follow broken behaviour unless -std=c99 is given?

Hapy new year too!
Jérôme.

Le 31/12/2011 16:37, Thomas Faber a écrit :
>   Hey,
>
>  On 2011-12-31 15:33, jgardou(a)svn.reactos.org wrote:
> >   Author: jgardou
> >  Date: Sat Dec 31 14:33:35 2011
> >  New Revision: 54793
> >
> >  URL: http://svn.reactos.org/svn/reactos?rev=54793&view=rev
> >  Log:
> >  [NEWINFLIB]
> >  - use _snwprintf instead of swprintf (MSVC is not compliant with standard here) 
> since _snwprintf seems unavailable on Unix, perhaps
>  -D_CRT_NON_CONFORMING_SWPRINTFS
>  would be an appropriate solution? That should cause VS2010 (2008 seems to have
>  the version without a length) to use the non-standard no-length version just
>  like GCC.
>  That [1] gets rid of the warnings for me.
>
>  Regards (and a happy new year :p),
>  Tom
>
>  [1] http://pastebin.com/4XU2SCXR
>
>  _______________________________________________
>  Ros-dev mailing list
>  Ros-dev(a)reactos.org
>  http://www.reactos.org/mailman/listinfo/ros-dev