search

complate / complate-stream

complate's core library for rendering HTML via JSX
https://complate.org
6 stars 7 forks source link

Non recursive processChildren #8

Closed tillsc closed 6 years ago

tillsc commented 6 years ago

There is no need to process non deferred children recursively. So I replaced it by a simple for. Why not something cool like forEach? Because it is not breakable.

FND commented 6 years ago

Thanks for this! It makes sense, but I haven't had a chance yet to really think it through - will do so soon.

Your commits conflicted with 12143392711b67eaceeff653541ab930178cf7bb (which I'd failed to push last week 😞 ) - I've resolved these conflicts in https://github.com/complate/complate-stream/compare/non-recursive-process-children?w=1 (plus minor stylistic tweaks).

FND commented 6 years ago

Merged as cc106c1 f. - thank you!