[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

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 analyzer for

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.

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 better, so I'm going