Re: #551: recipient handling of trailer fields

On Jan 30, 2014, at 6:44 PM, Bjoern Hoehrmann wrote:

> * Roy T. Fielding wrote:
>> and in the pseudo-code of 4.1.3 (Decoding Chunked):
>> 
>> OLD:
>>    read header-field
>>    while (header-field not empty) {
>>       append header-field to existing header fields
>>       read header-field
>>    }
>> 
>> NEW:
>>    read trailer field
>>    while (trailer field is not empty) {
>>       if trailer field is allowed to be sent in a trailer,
>>           append trailer field to existing header fields
>>       read trailer-field
>>    }
> 
> I think the combination of comma and indentation is difficult to under-
> stand. It would be better to be more explicit,
> 
>     read trailer field
>     while (trailer field is not empty) {
>        if (trailer field is allowed to be sent in a trailer) {
>            append trailer field to existing header fields
>        }
>        read trailer-field
>     }
> 
> for instance.

Fixed.  Cheers,

....Roy

Received on Saturday, 1 February 2014 10:04:13 UTC