module,src: do not wrap modules with -1 lineOffset

In b799a74 and dfee4e3 the module wrapping mechanism was changed for better error reporting. However, the changes causes issues with debuggers and profilers. This commit reverts the wrapping changes. Fixes: #4297 PR-URL: #4298 Reviewed-By: Ben Noordhuis <[email protected]> Reviewed-By: Ali Ijaz Sheikh <[email protected]>

Author: cjihrig

lib/module.js

@@ -372,7 +372,7 @@ Module.prototype._compile = function(content, filename){
 varwrapper=Module.wrap(content);
 
 varcompiledWrapper=runInThisContext(wrapper,
-{filename: filename,lineOffset: -1});
+{filename: filename,lineOffset: 0});
 if(global.v8debug){
 if(!resolvedArgv){
 // we enter the repl if we're not given a filename argument.

src/node.js

@@ -958,7 +958,7 @@
 };
 
 NativeModule.wrapper=[
-'(function (exports, require, module, __filename, __dirname){\n',
+'(function (exports, require, module, __filename, __dirname){',
 '\n});'
 ];
 
@@ -968,7 +968,7 @@
 
 varfn=runInThisContext(source,{
 filename: this.filename,
-lineOffset: -1
+lineOffset: 0
 });
 fn(this.exports,NativeModule.require,this,this.filename);
 

test/sequential/test-module-loading.js

@@ -281,9 +281,9 @@ assert.equal(42, require('../fixtures/utf8-bom.js'));
 assert.equal(42,require('../fixtures/utf8-bom.json'));
 
 // Error on the first line of a module should
-// have the correct line and column number
+// have the correct line number
 assert.throws(function(){
 require('../fixtures/test-error-first-line-offset.js');
 },function(err){
-return/test-error-first-line-offset.js:1:1/.test(err.stack);
+return/test-error-first-line-offset.js:1:/.test(err.stack);
 },'Expected appearance of proper offset in Error stack');