Skip to content
This repository has been archived by the owner on Sep 6, 2021. It is now read-only.

dependencies.js rewrite #9488

Merged
merged 1 commit into from
Oct 14, 2014
Merged

dependencies.js rewrite #9488

merged 1 commit into from
Oct 14, 2014

Conversation

TuurDutoit
Copy link
Contributor

I rewrote dependencies.js, to be faster with less code.

@peterflynn
Copy link
Member

Note: See original PR descrition in #9472:

The original code wasn't very optimized (8+ document.write calls!), so I rewrote it to be a little more performant.
Now I get an execution time of about 0.05ms, compared to 0.5ms before.

I just dived into Brackets' codebase, with the intention of optimizing some code (Brackets still feels a little slow), and this was an easy first pick.

@ingorichter
Copy link
Contributor

Looks good to me. Merging! Thanks.

ingorichter added a commit that referenced this pull request Oct 14, 2014
@ingorichter ingorichter merged commit 49d118f into adobe:master Oct 14, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants