New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Generate names field for identifiers to get correct names mappings #3658
Conversation
@@ -34,7 +34,7 @@ export default class SourceMap { | |||
* values to insert a mapping to nothing. | |||
*/ | |||
|
|||
mark(generatedLine: number, generatedColumn: number, line: number, column: number, filename: ?string) { | |||
mark(generatedLine: number, generatedColumn: number, line: number, column: number, identifierName: ?string, filename: ?string) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Getting max-len error here (do we still want this on), I'd be ok with removing as well)
Very cool, maybe a test for the sourcemap https://github.com/babel/babel/blob/master/packages/babel-generator/test/index.js if that's straightforward |
752d829
to
ee83dc1
Compare
ee83dc1
to
cc52e8b
Compare
👍 |
Current coverage is 89.69% (diff: 100%)@@ master #3658 diff @@
==========================================
Files 191 191
Lines 13582 17452 +3870
Methods 1423 2063 +640
Messages 0 0
Branches 3127 4739 +1612
==========================================
+ Hits 11981 15654 +3673
- Misses 1601 1798 +197
Partials 0 0
|
Caused an issue with webpack sourcemaps but was fixed - webpack/source-list-map#2 |
This allows things like name mangling to retain their original source location and allows debuggers to do realiasing etc.