avoid calling strlen() twice in readlink()

2012-03-08 Thread Václav Zeman
Hi. Here is a tiny patch to avoid calling strlen() twice in readlink(). ChangeLog: 2012-03-08 Václav Zeman * path.cc (readlink): Avoid calling strlen() twice. -- VZ --- path.cc 2012-03-07 18:10:44.0 +0100 +++ path.cc 2012-03-08 13:28:28.468266800 +0100 @@ -2782,7 +2

Re: avoid calling strlen() twice in readlink()

2012-03-08 Thread Corinna Vinschen
On Mar 8 14:37, Václav Zeman wrote: > Hi. > > Here is a tiny patch to avoid calling strlen() twice in readlink(). > > ChangeLog: > > 2012-03-08 Václav Zeman <...> > > * path.cc (readlink): Avoid calling strlen() twice. Thanks, applied. Corinna -- Corinna Vinschen

Re: avoid calling strlen() twice in readlink()

2012-03-08 Thread Eric Blake
On 03/08/2012 06:37 AM, Václav Zeman wrote: > Hi. > > Here is a tiny patch to avoid calling strlen() twice in readlink(). > > > - ssize_t len = min (buflen, strlen (pathbuf.get_win32 ())); > + size_t pathbuf_len = strlen (pathbuf.get_win32 ()); > + size_t len = MIN (buflen, pathbuf_len); >

Re: avoid calling strlen() twice in readlink()

2012-03-08 Thread Corinna Vinschen
On Mar 8 08:29, Eric Blake wrote: > On 03/08/2012 06:37 AM, Václav Zeman wrote: > > Hi. > > > > Here is a tiny patch to avoid calling strlen() twice in readlink(). > > > > > > > - ssize_t len = min (buflen, strlen (pathbuf.get_win32 ())); > > + size_t pathbuf_len = strlen (pathbuf.get_win32

Re: avoid calling strlen() twice in readlink()

2012-03-08 Thread Christian Franke
Corinna Vinschen wrote: On Mar 8 08:29, Eric Blake wrote: On 03/08/2012 06:37 AM, Václav Zeman wrote: Hi. Here is a tiny patch to avoid calling strlen() twice in readlink(). - ssize_t len = min (buflen, strlen (pathbuf.get_win32 ())); + size_t pathbuf_len = strlen (pathbuf.get_win32 ());