core-wg / href

Other
2 stars 0 forks source link

Reference resolution section: ambiguous prose #28

Closed thomas-fossati closed 2 years ago

thomas-fossati commented 2 years ago

In Section 5.3, third bullet, we say (note the last sentence):

"If the value of discard is true in the CRI reference, replace the path in the buffer with the empty array, unset query and fragment, and set a true authority to null. If the value of discard is an unsigned number, remove as many elements from the end of the path array; if it is non-zero, unset query and fragment. Set discard to true in the buffer."

It is not clear whether setting discard to true is triggered by matching the first condition ("If the value of discard is true in the CRI reference"), the second ("If the value of discard is an unsigned number"), or maybe even its sub-clause ("if it is non-zero").

cabo commented 2 years ago

This is meant to be unconditional. Discard is always true after step 3. I made a small edit to make this more obvious in #29