Improve CallSite.prototype.getFunctionName

[bnoordhuis]

Error.prepareStackTrace = function(s, cs) {
    print(cs[0].getFunctionName())
    return 42
}
try {
    eval("bad code")
} catch (_) {
    // ignore
}

Outputs:

$ qjs --script t.js 
null
<eval>

That is, the first call site has null as the function name. It'd be infinitesimally nicer if that also returned e.g. <eval>.

[saghul]

Node seems to do the same?

Error.prepareStackTrace = function(s, cs) {
    cs.forEach(csi => {
        console.log(csi.getFunctionName())
    });
    return 42
}
try {
    eval("bad code")
} catch (e) {
    console.log(e);
    // ignore
}

node ./t.cjs
null
Module._compile
Module._extensions..js
Module.load
Module._load
executeUserEntryPoint
null
[42]

[bnoordhuis]

Node seems to do the same?

Meaning we can do better :)

[saghul]

Haha true!