On Wed, May 21, 2014 at 05:07:36PM -0700, Kyle J. McKay wrote:
> >+void strbuf_tolower(struct strbuf *sb)
> >+{
> >+ size_t i;
> >+ for (i = 0; i < sb->len; i++)
> >+ sb->buf[i] = tolower(sb->buf[i]);
> >+}
> >+
>
> Wouldn't a direct transfer of the lowercase function be something more like:
>
>
> void strbuf_tolower(struct strbuf *sb)
> {
> char *p = sb->buf;
> for (; *p; p++)
> *p = tolower(*p);
> }
>
> That seems to me to be a bit more efficient. According to the comments in
> strbuf.c, "people can always assume buf is non NULL and ->buf is NUL
> terminated even for a freshly initialized strbuf."
Yes, and that would be fine with me (I actually wrote strbuf_tolower for
my own use, and _then_ realized that we already had such a thing that
could be replaced).
-Peff
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
