[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] procfs: another fix for the process file name in stat/status
From: |
Pino Toscano |
Subject: |
Re: [PATCH] procfs: another fix for the process file name in stat/status |
Date: |
Mon, 10 Sep 2012 17:31:32 +0200 |
User-agent: |
KMail/1.13.7 (Linux/3.2.0-3-amd64; KDE/4.8.4; x86_64; ; ) |
Alle domenica 9 settembre 2012, Samuel Thibault ha scritto:
> Pino Toscano, le Fri 07 Sep 2012 20:02:56 +0200, a écrit :
> > +static int args_filename_length (const char *name)
> > +{
> > + const char *p = name;
> > + while (*p != '\0' && *p != ' ')
> > + ++p;
>
> Why not using index(name, ' ') here?
If I would use strchr, the code would be something like:
const char *p = strchr (name, ' ');
return p != NULL ? p - name : strlen (name);
which, for the common case, would mean scanning the string twice.
If it is not considered an issue I can change it anyway.
--
Pino Toscano
signature.asc
Description: This is a digitally signed message part.