cscott/narcissus — comment on commit ee1e07b40c7a400061b8cc3ee815c099a8b3f7b5
File:  line 0
By olov:

I appreciate that you took the time to implement this, but I'm reluctant to diverging away further from the (outdated) Narcissus version we currently embed. A quick look at https://github.com/mozilla/narcissus/blob/master/lib/lexer.js seems to indicate that they have \r\n support there now. Also, David Herman has merged a number of fixes from other Narcissus users in the past months. Those will benefit us too.

We really want to get rebased as soon as possible. It's not easy given that our version of Narcissus is modified to produce a better (for Shaper's need) AST. If you're interested in participating with this, let me know. Otherwise I'll try to plan for an evening of rebasing fun.