Merge pull request #188 from johnjbarton/no-module-on-err

Don't create a Module if the evaluation fails.

Author: guybedford

lib/loader.js

@@ -807,7 +807,8 @@ function logloads(loads) {
       err = doExecute(module);
       if (err)
         module.failed = true;
-      module.module = _newModule(module.exports);
+      else
+        module.module = _newModule(module.exports);
       module.execute = undefined;
       return err;
     }
@@ -968,7 +969,7 @@ function logloads(loads) {
       // 26.3.3.14
       set: function(name, module) {
         if (!(module instanceof Module))
-          throw new TypeError('Set must be a module');
+          throw new TypeError('Loader.set(' + name + ', module) must be a module');
         this._loader.modules[name] = {
           module: module
         };