| 15:40 | <littledan> | I think the ends I’m looking for could be accomplished another way, then… |
| 15:40 | <littledan> | Okay, I just would have thought that the InnerModuleLinking logic repeats a lot of the payload logic you have. |
| 19:19 | <guybedford> | Usually when writing a module loader, you only need to do two passes over the graph |
| 19:19 | <guybedford> | one pass to load the modules, and another pass to execute them |
| 19:19 | <guybedford> | the specification currently already has two passes |
| 19:19 | <guybedford> | you're adding a third |
| 19:19 | <guybedford> | just wondering if we can condense that |
| 19:20 | <littledan> | it'd be nice if we could, but the three passes existed in ES+HTML before, didn't they? |
| 19:20 | <guybedford> | Yeah I guess once you add HTML that's three certainly |
| 19:21 | <littledan> | this patch is all about just moving the logic from HTML to ES without changing semantics |
| 19:21 | <littledan> | I honestly don't know how we could move linking to be on-line during fetching and parsing, but maybe you see a way |
| 19:22 | <guybedford> | that makes sense, I guess the distinction in the iteration approach is based on that async / sync difference as well |
| 19:22 | <littledan> | what do you mean by that? |
| 19:24 | <guybedford> | the InnerModuleLinking algorithm is a recursive functional approach, while the new logic carries a payload state and a continuation system. They're two ways of doing the same thing it seems? |
| 19:24 | <littledan> | oh, yes, that's related to being async |
| 19:24 | <littledan> | and the logic is really cribbed from HTML IIRC |
| 19:25 | <guybedford> | ah I see, didn't realise that |
| 19:26 | <guybedford> | This makes sense then! I wasn't suggesting anything major, was also just interested if there were other simplifications which result for InnerModuleLinking as we do this |
| 19:26 | <guybedford> | eg if parts of what is doing in InnerModuleLinking could be brought into the loading |
| 19:26 | <littledan> | maybe there are! I feel like getting all this in one place gives us more of an opportunity for that. |