search

maxtaco / coffee-script

IcedCoffeeScript
http://maxtaco.github.com/coffee-script
MIT License
726 stars 58 forks source link

SourceMaps doesn't work for included modules #71

Closed 0rland closed 8 years ago

0rland commented 11 years ago

It looks like line position of exceptions are mapped correctly for main module only. For required sources it uses line number of generated JS code.

maxtaco commented 11 years ago

This annoys me too. I'm waiting for it to be fixed in the CoffeeScript trunk and then will merge those changes over.

0rland commented 11 years ago

As I understand, it's already ok in original CoffeeScript 1.6.3, isn't it? Could you please look at diff. of versions of coffee-script.js files.

zapu commented 11 years ago

This change fixes it: https://github.com/jashkenas/coffee-script/commit/7f1088054c91f5ab3bf1ea1098b6ebffaa29a5a9

I realized I could just merge new coffeescript changes to iced. Please see the pull request. It also includes a change to sourcemap syntax.