Jim Meyering wrote:
> Looks useful.

It's useful in few cases, to get the ultimate performance. getc_unlocked is
already quite good in performance.

> >           const char *nl = (const char *) memchr (buf, '\n', n);
> 
> Surely you meant this:
> 
>             const char *nl = (const char *) memchr (buf, '\n', available);

Yes.

> >           if (nl != NULL)
> >             {
> >               obstack_grow (obs, buf, nl - buf);
> >               fseek (fp, nl - buf + 1, SEEK_CUR);

And here I forgot a 'break;'.

Bruno



Reply via email to