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.


Reply via email to