Re: [PATCH 1/6] lib/string: add strnchrnul()

From: Andy Shevchenko
Date: Sun Apr 28 2019 - 15:23:00 EST


On Sun, Apr 28, 2019 at 11:26:12AM -0700, Yury Norov wrote:
> On Sun, Apr 28, 2019 at 07:04:00PM +0300, Andy Shevchenko wrote:
> > On Sat, Apr 27, 2019 at 08:29:31PM -0700, Yury Norov wrote:
> > > New function works like strchrnul() with a length limited strings.
> >
> > The prototype better to be consistent with strchrnul() and memchr(), i.e.
> > the size parameter to go last.
>
> But if so it would be inconsistent with strnchr(). I'm OK to do the change,
> but I'm not sure if this version better than that.

Forgot to check it. In this case, yes, strnchr() + size.

--
With Best Regards,
Andy Shevchenko