Incorrect sourcemap for comments in module function

[NN---]

This works correctly, you cannot put a breakpoint on the comment:

function P() {
    // Test this
    var a = 1;
}

This is the sourcemap:

{"version":3,"file":"W.js","sourceRoot":"","sources":["W.ts"],"names":["P"],"mappings":"AAAA,SAAS,CAAC;IAETA,AADAA,YAAYA;QACRA,CAACA,GAAGA,CAACA,CAACA;AACXA,CAACA"}

While here you can put a breakpoint on the comment but cannot put on 'var a = 1' !

module Q {
    function P() {
        // Test this
        var a = 1;
    }
}

Sourcemap:

{"version":3,"file":"W.js","sourceRoot":"","sources":["W.ts"],"names":["Q","Q.P"],"mappings":"AAAA,IAAO,CAAC,CAKP;AALD,WAAO,CAAC,EAAC,CAAC;IACTA,SAASA,CAACA;QAETC,AADAA,YAAYA;YACRA,CAACA,GAAGA,CAACA,CAACA;IACXA,CAACA;AACFD,CAACA,EALM,CAAC,KAAD,CAAC,QAKP"}

Checked in Google Chrome.

[richardbullin]

I'm finding that the sourcemaps from versions 1.1.0-1 up to 1.4.1 are being generated incorrectly when I have a file with just a comment above a line without using a module. It also doesn't appear to matter if the comment is at the top or the middle of the source file.

In versions 1.1.0-1, 1.3.0 and 1.4.1 the below code with a breakpoint on 'var aNumber = 10;' does not get hit

// This is a comment
var aNumber = 10;

var aNumber2 = 1100;

The generated sourcemap for all of these versions is

{"version":3,"file":"_CompiledJavaScript.js","sourceRoot":"","sources":["../TestMain.ts"],"names":[],"mappings":"AACA,AADA,oBAAoB;IAChB,OAAO,GAAG,EAAE,CAAC;AAEjB,IAAI,QAAQ,GAAG,EAAE,CAAC"}

Compared to the sourcemap for the same code generated from 1.0.1 which does allow the breakpoint to get hit

{"version":3,"file":"_CompiledJavaScript.js","sourceRoot":"","sources":["../TestMain.ts"],"names":[],"mappings":"AAAA,oBAAoB;AACpB,IAAI,OAAO,GAAG,EAAE;;AAEhB,IAAI,QAAQ,GAAG,EAAE"}

[mhegazy]

@yui, Sheetal had a fix for this a while back; so check #2014 first.