Re: Last Call: <draft-nottingham-rfc5988bis-05.txt> (Web Linking) to Proposed Standard

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> On 15 May 2017, at 11:42 am, Mark Nottingham <mnot@xxxxxxxx> wrote:
> 
>>>> 
>>>> 1.  Let "links" be an empty list.
>>>> 
>>>> 2.  Create "link_strings" by splitting "field_value" on ","
>>>>     characters, excepting "," characters within quoted strings as per
>>>> 
>>>> I have my doubts that people will be able to translate it into actually correct code. To detect whether something is inside a quoted-string essentially requires running a parser; the prose here suggests that this is not necessary.
>>> 
>>> I reluctantly agree. I'll need to rework the algorithm to address this; please stand by.
>> 
>> The only way to properly split list supporting field values is to parse them according to the grammar for the list items; I don't believe in the existence of a shortcut that actually works correctly. Suggesting otherwise IMHO will lead to broken implementations for certain edge cases.
> 
> +1
> 
> 
>>>> It also should mention that there might be multiple instances of field_value.
>>> 
>>> The algorithm is for parsing a single field-value...
>> 
>> Yes, but should we tell the audience that they need to parse *all* field values, and combine the result?
> 
> OK.

This is what I currently have:
  https://gist.github.com/mnot/2fb569e7303dbcdde8b27cb7a404a648

I need to get some more eyeballs onto it to make sure it's correct (or at least reasonable). Reviews appreciated. 

Cheers,


--
Mark Nottingham   https://www.mnot.net/





[Index of Archives]     [IETF Annoucements]     [IETF]     [IP Storage]     [Yosemite News]     [Linux SCTP]     [Linux Newbies]     [Fedora Users]