kitsonk/dojo — comment on commit ba9465fcd1207b12a056a8f91ff76f27d976c6f9
File: parser.js line 586
By neonstalwart:

i'd be interested to see which @wkeese prefers. you could simplify this whole block by avoiding checking for `mids.length`. it means you would be always calling require (even for an empty list) but it would return synchronously when the list is empty or all modules are already loaded so not sure if that matters.

```js
require(mids, function () {
idx = 0;
while(obj = objects[idx++]) {
obj.ctor = _getCtor(obj.type, obj.mixins);
obj.proto = obj.ctor && obj.ctor.prototype;
}
mapCtorDeferred.resolve(objects);
});
return mapCtorDeferred;
```

this would also remove the need for `mapObjects` since the code is inline in one place. overall, this is probably more a matter of personal preference so that's why i'd be interested in what @wkeese thinks.