Skip to content

Avoid a crash when no raw#346

Open
ef4 wants to merge 1 commit into
michaelficarra:masterfrom
ef4:missing-raw
Open

Avoid a crash when no raw#346
ef4 wants to merge 1 commit into
michaelficarra:masterfrom
ef4:missing-raw

Conversation

@ef4

@ef4 ef4 commented May 27, 2015

Copy link
Copy Markdown
Contributor

You can make the compiler crash by trying to compile this file:

argv[2..-1].join()

like this:

/bin/coffee --js --input sample.coffee --source-map-file sample.map

This change defends against the missing expression.raw, since I'm inferring from the surrounding code that raw is not always present on every node.

@michaelficarra

Copy link
Copy Markdown
Owner

We should fix the accidentally missing raw field.

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

Successfully merging this pull request may close these issues.

2 participants