Re: [PATCH] Polish helper debugging

On 12/14/2012 01:38 AM, Amos Jeffries wrote:
> On 14/12/2012 9:15 p.m., Kinkie wrote:
>> On Fri, Dec 14, 2012 at 2:00 AM, Alex Rousskov
>> <rousskov_at_measurement-factory.com> wrote:
>>> Hello,
>>>
>>> The attached patch polishes helper debugging. These changes were
>>> inspired by production 3.0 code where support folks apparently found
>>> them useful. They add message length and/or message contents debugging
>>> to several helper processing paths. I think they are worth accepting.
>>>
>>> I ported and rewrote the changes using the Raw manipulator proposed in
>>> the previous patch.
>> I haven't seen the previous patch, but this looks good to me. +1.
>>
>
>
> +1. mee too. ;-)

By the "previous patch", I meant Raw patch posted just before the helper
debugging patch. Sorry for being unclear.

Committed to trunk as r12523.

Thank you,

Alex.
Received on Fri Dec 14 2012 - 16:07:43 MST