Re: [PATCH] msvc: Directly use MS version (_stricmp) of strcasecmp
- Date: Sun, 18 Nov 2018 16:58:20 -0500
- From: Jeff King <peff@xxxxxxxx>
- Subject: Re: [PATCH] msvc: Directly use MS version (_stricmp) of strcasecmp
On Sun, Nov 18, 2018 at 10:02:02PM +0100, Sven Strickroth wrote:
> This also removes an implicit conversion from size_t (unsigned) to int (signed).
> _stricmp as well as _strnicmp are both available since VS2012.
Once upon a time we had problems with taking a function pointer of
strcasecmp (to use as a comparator with string_list), so I wondered if
that might be part of why it's defined the way it is.
But the current definition is already inline:
> -static __inline int strcasecmp (const char *s1, const char *s2)
> - int size1 = strlen(s1);
> - int sisz2 = strlen(s2);
> - return _strnicmp(s1, s2, sisz2 > size1 ? sisz2 : size1);
> +#define strcasecmp _stricmp
And it seems we worked around this in de2f95ebed (mailmap: work around
implementations with pure inline strcasecmp, 2013-09-12). So I don't
think there is any blocker there.
(Though of course I have no idea on other portability questions around
_stricmp(); I'll leave that for Windows folks).