You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@httpd.apache.org by "William A. Rowe, Jr." <wr...@rowe-clan.net> on 2006/10/15 21:02:01 UTC

Re: svn commit: r464141 - in /httpd/httpd/trunk/srclib/pcre: dftables.c dftables.dsp

> -# ADD CPP /nologo /MD /W3 /O2 /D "_WIN32" /D "NDEBUG" /D "_CONSOLE" /D "_MBCS" /Fd"Release\dftables" /FD /c
> +# ADD CPP /nologo /MD /W3 /wd:4996 /O2 /D "_WIN32" /D "NDEBUG" /D "_CONSOLE" /D "_MBCS" /Fd"Release\dftables" /FD /c
>  # ADD BASE RSC /l 0x809 /d "NDEBUG"
>  # ADD RSC /l 0x809 /d "NDEBUG"
>  BSC32=bscmake.exe
> @@ -68,7 +68,7 @@
>  # PROP Ignore_Export_Lib 0
>  # PROP Target_Dir ""
>  # ADD BASE CPP /nologo /MDd /W3 /EHsc /Zi /Od /D "_WIN32" /D "_DEBUG" /D "_CONSOLE" /D "_MBCS" /Fd"Debug\dftables" /FD /c
> -# ADD CPP /nologo /MDd /W3 /EHsc /Zi /Od /D "_WIN32" /D "_DEBUG" /D "_CONSOLE" /D "_MBCS" /Fd"Debug\dftables" /FD /c
> +# ADD CPP /nologo /MDd /W3 /wd:4996 /EHsc /Zi /Od /D "_WIN32" /D "_DEBUG" /D "_CONSOLE" /D "_MBCS" /Fd"Debug\dftables" /FD /c

why not s#/wd:4996#/D "_CRT_SECURE_NO_DEPRECATE"#

in the .dsp, provided it solves all of the emits?

Re: svn commit: r464141 - in /httpd/httpd/trunk/srclib/pcre: dftables.c dftables.dsp

Posted by Mladen Turk <mt...@apache.org>.
William A. Rowe, Jr. wrote:
> Mladen Turk wrote:
>> William A. Rowe, Jr. wrote:
>>> why not s#/wd:4996#/D "_CRT_SECURE_NO_DEPRECATE"#
>>>
>> /wd with colon like (/wd:4966) gives:
>> Command line error D8021 : invalid numeric argument '/wd:4996'    cl
>> when converted to VS2005.
>> BTW according to the MSDN the proper format is /wdNNNN not /wd:NNNN
> 
> Yes - I understood that from your first commit message, thank you for this
> catch!
> 
>> Also the _CRT_SECURE_NO_DEPRECATE is not required.
> 
> Since you proposed that define as the alternative to /wd4996, I was curious
> why you preferred to disable the warning over defining out the warning?
> That's all - either way thank you for the fix.
> 

It was a simple copy/paste from apr.hw,
so that's why _CRT_SECURE_NO_DEPRECATE get into focus :)
VS2005 compiles the converted dftables.vcproj without warnings.

Regards,
Mladen.

Re: svn commit: r464141 - in /httpd/httpd/trunk/srclib/pcre: dftables.c dftables.dsp

Posted by "William A. Rowe, Jr." <wr...@rowe-clan.net>.
Mladen Turk wrote:
> William A. Rowe, Jr. wrote:
>>
>> why not s#/wd:4996#/D "_CRT_SECURE_NO_DEPRECATE"#
>>
> 
> /wd with colon like (/wd:4966) gives:
> Command line error D8021 : invalid numeric argument '/wd:4996'    cl
> when converted to VS2005.
> BTW according to the MSDN the proper format is /wdNNNN not /wd:NNNN

Yes - I understood that from your first commit message, thank you for this
catch!

> Also the _CRT_SECURE_NO_DEPRECATE is not required.

Since you proposed that define as the alternative to /wd4996, I was curious
why you preferred to disable the warning over defining out the warning?
That's all - either way thank you for the fix.



Re: svn commit: r464141 - in /httpd/httpd/trunk/srclib/pcre: dftables.c dftables.dsp

Posted by Mladen Turk <mt...@apache.org>.
William A. Rowe, Jr. wrote:
> 
> why not s#/wd:4996#/D "_CRT_SECURE_NO_DEPRECATE"#
>

/wd with colon like (/wd:4966) gives:
Command line error D8021 : invalid numeric argument '/wd:4996'	cl
when converted to VS2005.
BTW according to the MSDN the proper format is /wdNNNN not /wd:NNNN

Also the _CRT_SECURE_NO_DEPRECATE is not required.

Regards,
Mladen.

Re: svn commit: r464141 - in /httpd/httpd/trunk/srclib/pcre: dftables.c dftables.dsp

Posted by Mladen Turk <mt...@apache.org>.
William A. Rowe, Jr. wrote:
> 
> why not s#/wd:4996#/D "_CRT_SECURE_NO_DEPRECATE"#
>

/wd with colon like (/wd:4966) gives:
Command line error D8021 : invalid numeric argument '/wd:4996'	cl
when converted to VS2005.
BTW according to the MSDN the proper format is /wdNNNN not /wd:NNNN

Also the _CRT_SECURE_NO_DEPRECATE is not required.

Regards,
Mladen.