jrburke/requirejs — pull request #418 by maxlk (2012-08-16 13:26:09+00:00)
Title: Relates to #417, check each module only once during one `findCycle()` search

Now `findCycle()` stores modules that were already checked in `processed` variable, and doesn't check them twice. So now it works like classic depth-first search - significantly more effective.

Avoiding usage of `mixin()` also improves performance.

I'm not sure about this:
``` javascript
if (!depMod.inited || !depMod.enabled) {
//Dependency is not inited, so this cannot
//be used to determine a cycle.
return true;
}
```
Now it returns `true` and aborts all search (not only current dependency). Is it desired?