Subject: Re: bug#18777: [PATCH] dfa: improvement for checking of multibyte character boundary

From: Norihiro Tanaka <[email-masked]>
Date: Wed, 17 Dec 2014 08:22:20 +0900

On Tue, 16 Dec 2014 09:12:21 -0800
Paul Eggert <[email-masked]> wrote:
> 
> This part of the patch does too much work, as the caller inspects *WCP
> only when skip_remains_mb returns a value not equal to p.  So there's
> no need for the "wcp == NULL &&" test in the patch. Instead, the
> documented API can change, saying that *WCP is assigned to only if WCP
> is non-NULL and the result is greater than p.

Thanks, you are right.  However, first it is no longer portable after
remove it.  Second if it is compiled with GCC 4.3 or later, the function
is inlined by and "WCP == NULL &&" will be pruned.