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

Reply via email to