Re: [whatwg/dom] Stop using “strictly split” in “validate and extract” (PR #1455)

sideshowbarker left a comment (whatwg/dom#1455)

For the record here, what _“validate and extract”_ said before the patch associated with https://github.com/whatwg/dom/issues/1134 was this:

> If _qualifiedName_ contains a U+003A (:), then [strictly split](https://infra.spec.whatwg.org/#strictly-split) the string on it and set _prefix_ to the part before and _localName_ to the part after.

So, it was ambiguous — as https://github.com/whatwg/dom/issues/1134 pointed out. But, one of the two ways it _could_ have been read as intending is:

> If _qualifiedName_ contains a U+003A (:), then <strike>[strictly split](https://infra.spec.whatwg.org/#strictly-split) the string on it and</strike> set _prefix_ to the part before and _localName_ to the part after.

In other words, it _could_ have been read as actually intending what the change in this PR does. In other-other words, the change here is just reverting the spec back to what it had actually  originally been intended to specify.

And nobody in the https://github.com/whatwg/dom/issues/1134 discussion ever explicitly instead said anything like, _“**Throw away everything else in the name** after splitting it into a list and taking just the first two list items is clearly what we had meant to require here.”_

Instead, that’s just… effectively what it had ended up getting changed to state — without anybody ever having explicitly said there was some good reason for it needing to require throwing away everything else in the name.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/dom/pull/1455#issuecomment-4014371682
You are receiving this because you are subscribed to this thread.

Message ID: <whatwg/dom/pull/1455/c4014371682@github.com>