Skip to content
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

Add metadata for source maps #6

Open
dominiccooney opened this issue Sep 27, 2018 · 2 comments
Open

Add metadata for source maps #6

dominiccooney opened this issue Sep 27, 2018 · 2 comments

Comments

@dominiccooney
Copy link
Member

JavaScript embeds sourcemaps with magic comments like //# sourceMappingURL=/path/file.js.map. BinAST files will also need metadata like this.

BinAST may also need to amend the sourcemap format, because BinAST files do not have line and column numbers or, for that matter, even character-aligned data.

@sendilkumarn
Copy link

This means the sourcemap will be in a separate format & rules like .binmap
or can we just decode binast into js and use the sourcemap w.r.t js if that is easier & faster to do. (anyways it will be mostly used only in the development)

@Yoric
Copy link
Contributor

Yoric commented Oct 3, 2018

I like @sendilkumarn's idea. If we can somehow generate a sourcemap from the decoded binast, we don't need to amend the sourcemap format. Can we?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants