Use of strcasestr
Dirk Hohndel
hohndel at infradead.org
Mon Apr 12 21:09:50 PDT 2010
On Tue, 13 Apr 2010 04:04:39 +0200, Tomas Carnecky <tom at dbservice.com> wrote:
> On 4/12/10 10:18 PM, Mikhail Gusarov wrote:
> >
> > Twas brillig at 15:58:10 12.04.2010 UTC+02 when tom at dbservice.com did gyre and gimble:
> >
> > TC> In 4fd9ea0 (guess From address from Received headers, 2010-04-06) you introduced
> > TC> strcasestr, which is not portable, see 82e47ec (notmuch reply: Use strstr
> > TC> instead of strcasestr for portability., 2010-02-04).
> >
> > TC> Is strcasestr really necessary there or can it be replaced with strstr?
> >
> > strcasecmp is POSIX.1-2001.
>
> Indeed it is, but the code uses strcasestr and I couldn't find any
> indication which standard that function is part of.
>
> Adding that function to compat/ probably is the way to go, but the whole
> compat mechanism doesn't work here. It's like if compat/Makefile.local
> was not included in the top-level makefile, notmuch_compat_srcs is empty
> there. Any ideas how to debug that?
While I don't have access to a system that doesn't provide strcasest
right now... I'll submit a patch in a moment that should add a
replacement function to compat.
Please check and make sure this works...
Thanks
/D
--
Dirk Hohndel
Intel Open Source Technology Center
More information about the notmuch
mailing list