On Mon, Apr 9, 2007, Paul J Stevens <[EMAIL PROTECTED]> said:

> Paul J Stevens wrote:
>> Aaron Stone wrote:
>>> I bet bug #565 is the same as I saw this morning. Possibly fixed in SVN,
>>> but I didn't have time to read the code fully. In dbmail-imapsession.c,
>>> _imap_session_fetch_parse_octet_range:
>>>
>>>         if (! token)
>>> -               return self->args_idx;
>>> +               return 0;
>>> +//FIXME wrong return value?       return self->args_idx;
>>>
>>> Paul, could you verify if this is correct behavior? It works for me now,
>>> just want to make sure I didn't kill some other functionality.
>> 
>> That doesnt look right at all. I'll have to look at this later.
> 
> Ok. Looks like return 0 is valid after all.

When / why did this code or its consumers change? I wonder if there was
something else that you were trying to fix that is now broken again and
needs to be fixed with some other approach.

Aaron
_______________________________________________
Dbmail-dev mailing list
[email protected]
http://twister.fastxs.net/mailman/listinfo/dbmail-dev

Reply via email to