Skip to content

Defensive coding to prevent out-of-range column #138

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

Merged
merged 1 commit into from
Aug 17, 2017

Conversation

Yogu
Copy link
Contributor

@Yogu Yogu commented Jun 1, 2016

There may be errors on line 1 column 1 and those resulted in a column of -62 which threw an out-of-range error.

Our example was: "v8debug is not defined" which can occur when debugging a node app from WebStorm

There may be errors on line 1 column 1 and those resulted in a column of -62 which threw an out-of-range error.
@basarat
Copy link

basarat commented Sep 5, 2016

I am getting this as well whenever the stack points to line 1 : #148 Thanks for the fix. Need it for alm-tools/alm#181 🌹

@basarat basarat mentioned this pull request Sep 6, 2016
29 tasks
@robertohuertasm
Copy link

robertohuertasm commented Aug 17, 2017

@evanw is this going to be merged any time soon? @angular/cli has introduced a dependency with your library and makes electron apps not work when testing. It would be nice if you could accept this PR. 😉

@LinusU
Copy link
Collaborator

LinusU commented Aug 17, 2017

Restarting Travis, I'll merge as soon as the test goes green 👍

@LinusU
Copy link
Collaborator

LinusU commented Aug 17, 2017

Hmm, Travis is failing, don't know if it's related to this PR, or just that it runs with Node.js 8.4 now (last test was run with 8.1.x)

@LinusU
Copy link
Collaborator

LinusU commented Aug 17, 2017

Testing out in #187

@LinusU LinusU merged commit 9addc16 into evanw:master Aug 17, 2017
@LinusU
Copy link
Collaborator

LinusU commented Aug 17, 2017

Thanks!

@LinusU
Copy link
Collaborator

LinusU commented Aug 17, 2017

Released as 0.4.16

@robertohuertasm
Copy link

Thanks @LinusU!! Really appreciate it!

@Yogu Yogu deleted the fix-out-of-range-columns branch August 20, 2017 21:11
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.

4 participants