Re: [PATCH] cwrapper: avoid duplicate strlen calculation.

2012-01-30 Thread Peter Rosin
Bob Friesenhahn skrev 2012-01-30 18:11: > On Mon, 30 Jan 2012, Peter Rosin wrote: > >> [Sorry for replying to myself] > > I do that often. > >>> * build-aux/ltmain.m4sh (func_emit_cwrapperexe_src:lt_update_exe_path): >>> Remove duplicate strlen calculation. >> >> Hmmm, I like the following bette

Re: [PATCH] cwrapper: avoid duplicate strlen calculation.

2012-01-30 Thread Bob Friesenhahn
On Mon, 30 Jan 2012, Peter Rosin wrote: [Sorry for replying to myself] I do that often. * build-aux/ltmain.m4sh (func_emit_cwrapperexe_src:lt_update_exe_path): Remove duplicate strlen calculation. Hmmm, I like the following better, so I'm going to push that instead, in case of silence. W

Re: [PATCH] cwrapper: avoid duplicate strlen calculation.

2012-01-30 Thread Peter Rosin
[Sorry for replying to myself] Peter Rosin skrev 2012-01-30 15:32: > Hi! > > This is very close to obvious and I very nearly just pushed it out, but > what's the rush? However, I will push under the 72h rule if noone > speaks up. > > It fixes half the issues reported by the clang static analyze

[PATCH] cwrapper: avoid duplicate strlen calculation.

2012-01-30 Thread Peter Rosin
Hi! This is very close to obvious and I very nearly just pushed it out, but what's the rush? However, I will push under the 72h rule if noone speaks up. It fixes half the issues reported by the clang static analyzer for my current pet project. Cheers, Peter >From a2e87f5bfed174de7d715dcf5ca5b5