This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [patch][committed] Fix second instance of BZ #18043

From: Paul Eggert <eggert at cs dot ucla dot edu>
To: Paul Pluzhnikov <ppluzhnikov at google dot com>
Cc: GLIBC Devel <libc-alpha at sourceware dot org>
Date: Mon, 09 Mar 2015 14:08:44 -0700
Subject: Re: [patch][committed] Fix second instance of BZ #18043
Authentication-results: sourceware.org; auth=none
References: <CALoOobM_bLg9CtR5vF+nfnhVtoZhKKo8extbCwAwmkNAPOSVLg at mail dot gmail dot com> <54FDE447 dot 4040000 at cs dot ucla dot edu> <CALoOobPHBaKhgOp8tmUe=m68uHNkSLjBOCrGwVaPp4co-4O3Hg at mail dot gmail dot com>

On 03/09/2015 11:58 AM, Paul Pluzhnikov wrote:

+#define CHAR_IN_SET(ch, char_set) \
+  (memchr (char_set "", (ch), sizeof (char_set) - 1) != NULL)

Yes, thanks, that looks good, except the parentheses are redundant in"(ch)".

Follow-Ups:
- Re: [patch][committed] Fix second instance of BZ #18043
  - From: Paul Pluzhnikov

References:
- [patch][committed] Fix second instance of BZ #18043
  - From: Paul Pluzhnikov
- Re: [patch][committed] Fix second instance of BZ #18043
  - From: Paul Eggert
- Re: [patch][committed] Fix second instance of BZ #18043
  - From: Paul Pluzhnikov

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]