summaryrefslogtreecommitdiff
path: root/test/fixtures/test-error-first-line-offset.js
AgeCommit message (Collapse)Author
2017-07-25test: replace CRLF by LF in a fixtureVse Mozhet Byt
test/fixtures/test-error-first-line-offset.js is used in the test/sequential/test-module-loading.js, but the line break type is insignificant. It can be replaced for consistency. PR-URL: https://github.com/nodejs/node/pull/14437 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Refael Ackermann <refack@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Timothy Gu <timothygu99@gmail.com>
2015-11-20module: fix column offsets in errorsTristian Flanagan
Because Node modules are wrapped, errors on the first line of a file leak the wrapper to the user and report the wrong column number. This commit adds a line break to the module wrapper so that the first line is treated the same as all other lines. To compensate for the additional line, a line offset of -1 is also applied to errors. Fixes: https://github.com/nodejs/node/issues/2860 PR-URL: https://github.com/nodejs/node/pull/2867 Reviewed-By: Colin Ihrig <cjihrig@gmail.com>