On 01/19/2011 05:44 PM, Dhammika Pathirana wrote:

>>> +            //  Check if we had an error in previous attempt.
>>> +            if (unlikely (!(static_cast<T*>    (this)->next)))
>>> +                return (size_t) -1;
>>> +
>>
>> I think this won't work. The next step of the decoding state machine is
>> performed here irrespective of whether previous step was fully processed or
>> not.
>>
> That would be something like this->*next ()
> We're checking this->next pointer for null, not dereferencing it.

Ugh. You are right. Sorry.

Thijs, does this patch solve your problem?

Martin
_______________________________________________
zeromq-dev mailing list
zeromq-dev@lists.zeromq.org
http://lists.zeromq.org/mailman/listinfo/zeromq-dev

Reply via email to