From 5981fb7faa13de95550b01272ebfbd8a5220aadb Mon Sep 17 00:00:00 2001 From: Michaël Zasso Date: Wed, 28 Aug 2019 10:05:32 +0200 Subject: assert: fix line number calculation after V8 upgrade MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit PR-URL: https://github.com/nodejs/node/pull/29694 Reviewed-By: Colin Ihrig Reviewed-By: Anna Henningsen Reviewed-By: Gus Caplan Reviewed-By: Jiawen Geng Reviewed-By: Michaël Zasso Reviewed-By: Tobias Nießen Reviewed-By: Ujjwal Sharma --- lib/assert.js | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) (limited to 'lib') diff --git a/lib/assert.js b/lib/assert.js index a83dba770e..d307582d1f 100644 --- a/lib/assert.js +++ b/lib/assert.js @@ -269,7 +269,7 @@ function getErrMessage(message, fn) { const call = err.stack[0]; const filename = call.getFileName(); - const line = call.getLineNumber() - 1; + let line = call.getLineNumber() - 1; let column = call.getColumnNumber() - 1; let identifier; let code; @@ -289,6 +289,9 @@ function getErrMessage(message, fn) { return message; } code = String(fn); + // For functions created with the Function constructor, V8 does not count + // the lines containing the function header. + line += 2; identifier = `${code}${line}${column}`; } -- cgit v1.2.3