On 07/24/2013 08:15 AM, Scott Talbert wrote: > On Wed, 24 Jul 2013, Phil Dibowitz wrote: > >>> Actually, it looks like 'len' is used elsewhere, so we can't get rid of >>> that. I'll send a new patch with the commented lines deleted and the >>> commit message reformatted. >> >> Huh? The function ends there - there's a return statement... and you're >> removing the only line that uses it... > > It's a pass-by-reference parameter.
Ah. that'll teach me to just look at the patch. BTW, if you're at OSCON come say hi... :) - Phil ------------------------------------------------------------------------------ See everything from the browser to the database with AppDynamics Get end-to-end visibility with application monitoring from AppDynamics Isolate bottlenecks and diagnose root cause in seconds. Start your free trial of AppDynamics Pro today! http://pubads.g.doubleclick.net/gampad/clk?id=48808831&iu=/4140/ostg.clktrk _______________________________________________ concordance-devel mailing list concordance-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/concordance-devel