| 09:25 | <evilpie> | Can someone help me understand how https://infra.spec.whatwg.org/#list-sort-in-ascending-order works. Does it return the sorted list or not? |
| 09:26 | <evilpie> | It creates a new list sorted, but it doesn't seem like it's being returned? |
| 09:30 | <Noam Rosenthal> | It constructs a new list... usually constructors don't "return" the created object. A lot of infra algos are like that (e.g. https://infra.spec.whatwg.org/#list-clone) |
| 09:33 | <evilpie> | So it's kinda like in place sort? https://html.spec.whatwg.org/#collect-tags-from-speculative-load-candidates uses it in that fashion. |
| 09:34 | <Noam Rosenthal> | no, to be clear the result of the algo a new list. But the algo does not use "return" |
| 09:35 | <Noam Rosenthal> | it's like new List(old_list.ascending_sorted_items) |
| 09:35 | <Noam Rosenthal> | There is no "return" here, but the result is equivalent to returning something |
| 09:39 | <evilpie> | So would a correct use look like "Set tags to sort in acending order tags" or does "Sort in acending order tags" also implicitly mean that? |
| 09:40 | <Noam Rosenthal> | If it points to that algorithm, it specifically creates a new list |
| 09:40 | <evilpie> | (It's pretty obvious what the result should be of course, just nitpicking here) |
| 09:41 | <Noam Rosenthal> | yea it's a bit awkward... I read https://html.spec.whatwg.org/#collect-tags-from-speculative-load-candidates as
|
| 09:43 | <Noam Rosenthal> | e.g. in other uses like https://fetch.spec.whatwg.org/#convert-header-names-to-a-sorted-lowercase-set it's more explicit |
| 09:46 | <Noam Rosenthal> | The use in https://html.spec.whatwg.org/multipage/nav-history-apis.html#windowproxy-getownproperty also makes it look like a sort-in-place. But in both those usages (this and speculative-load) there should not be an observable difference between sorting in place and creating a new list |
| 09:54 | <evilpie> | Yeah. But it confused me, because I couldn't understand which way it actually works. |
| 10:04 | <zcorpan> | Sam Sneddon [:gsnedders]: ping https://github.com/web-platform-tests/rfcs/pull/226 |
| 11:59 | <annevk> | evilpie: I think you should file an issue. It should return the newly created list I think. I don't think it's a constructor (IDL has specific logic for "constructor steps", but those don't apply to everyday algorithms, just "constructor steps"). |
| 13:41 | <cwilso> | Hey gang - I'm not seeing any topics added to this Thursday's meeting yet; if it does happen, I will need someone else to chair as I'll be on a plane at that time. Any takers? |
| 13:55 | <Noam Rosenthal> | There are topics with Agenda+ that we didn't get to last time |
| 13:56 | <cwilso> | ah, right you are. I hadn't added the new block to the agenda doc yet |
| 13:56 | <Noam Rosenthal> | Also, I can chair again |
| 13:57 | <cwilso> | thanks! |
| 13:57 | <Noam Rosenthal> | cwilso: do you mind doing the rest of the preliminary work, and I'll chair the actual meeting? |
| 14:07 | <cwilso> | Not at all, I'll update the agenda doc |
| 14:08 | <cwilso> | Oh, I see you have already. :P |
| 14:13 | <Noam Rosenthal> | (I just did the carryovers, not sure what else is needed like deal with the WHATWG github issue for the meeting etc) |
| 20:26 | <hacknorris> | where to put html ideas? |
| 20:26 | <hacknorris> | just asking 🙂 |