On Tue, 16 Dec 2014 09:12:21 -0800 Paul Eggert <egg...@cs.ucla.edu> 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.
