POSIX function such as stricmp and strnicmp are deprecated beginning in Visual C++ 2005. Use ISO C++ conformant _stricmp and _strnicmp instead in StringExtras.h.
Created attachment 41350 [details]
Use ISO C++ conformant _stricmp and _strnicmp
darin, arboen or sfalken are likely better reviewers for this than I.
The bug title, ChangeLog, and Visual Studio warning are incredibly misleading. strnicmp is not part of POSIX and _strnicmp is certainly not part part of the ISO C++ standard. They’re both Microsoft extensions. The change itself is probably fine but IMO the ChangeLog should be corrected.
Change the title.
Created attachment 41354 [details]
Update the ChangeLog because stricmp and strnicmp are not POSIX functions but Microsfot extensions.
Comment on attachment 41354 [details]
Clearing flags on attachment: 41354
Committed r49765: <http://trac.webkit.org/changeset/49765>
All reviewed patches have been landed. Closing bug.