Date: Mon, 18 Mar 2013 21:18:30 -0400 From: <jfoug@....net> To: john-dev@...ts.openwall.com Cc: magnum <john.magnum@...hmail.com> Subject: Re: basename portability 'fix'. ---- magnum <john.magnum@...hmail.com> wrote: > Parts of your patch has CRLF and parts of it does not. This made git disregard most of it as "whitespace errors" :-) Also, I get this: > > misc.c: In function 'jtr_basename_r': > misc.c:199:19: warning: variable 'name' set but not used [-Wunused-but-set-variable] > misc.c: In function 'jtr_basename_r': > misc.c:199:19: warning: variable 'name' set but not used [-Wunused-but-set-variable] > > I guess we can just drop all statements that declare or set name? Yes, I believe so. I originally used it, before adding the "if (p && !SEP_CHAR(p))" code within the while statement. It was used walk backwards, since the base pointer would walk off the end of the string in cases like: /usr/local/ Now, base will be 'local/' since I do not walk the base pointer, until there is something 'useful' past the current part. So, yes, we can drop the var, and it's initialization, it is not needed any more. Also, the param _name can simply be changed to name like Lukas mentioned. Jim.
Powered by blists - more mailing lists
Powered by Openwall GNU/*/Linux - Powered by OpenVZ