Re: [whatwg/url] Editorial: adopt Infra number definitions (and redo IPv6 pieces) (PR #838)

@domenic commented on this pull request.



> + <li><p>Let <var>compress</var> be null.
+
  <li>
-  <p>Let <var>compress</var> be an index to the first <a>IPv6 piece</a> in the first longest
-  sequences of <var>address</var>'s <a>IPv6 pieces</a> that are 0.
+  <p>If multiple <var>address</var>'s <a for="IPv6 address">pieces</a> are 0 sequentially, then set
+  <var>compress</var> to the index of the first item of the first of the longest sequences of
+  <var>address</var>'s <a for="IPv6 address">pieces</a> that are 0.
 
   <p class=example id=example-e2b3492e>In <code>0:f:0:0:f:f:0:0</code> it would point to
   the second 0.
 
- <li><p>If there is no sequence of <var>address</var>'s <a>IPv6 pieces</a> that are 0 that is
- longer than 1, then set <var>compress</var> to null.
-
  <li><p>Let <var>ignore0</var> be false.

I think this changed from excluding 1-length sequences to including them.

FYI here is whatwg-url's implementation. It might be worth writing the whole thing out as it's pretty complex. https://github.com/jsdom/whatwg-url/blob/0fb65e3c3671c2e790ff38aed9332fc7fb2cd12b/lib/url-state-machine.js#L387

-- 
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/url/pull/838#discussion_r1849367457
You are receiving this because you are subscribed to this thread.

Message ID: <whatwg/url/pull/838/review/2447138115@github.com>