non-standard properties on Function prototype

[p-bakker]

Looks like there are some non-standard properties on the function prototype, like columnNumber, lineNumber and fileName

Do they serve a purpose?

[p-bakker]

And Set seems to have a non-standard .groupBy method

[saghul]

This is a bug because Set and Map reuse the class functions, we can fix that, thanks for the report!

[saghul]

Looks like there are some non-standard properties on the function prototype, like columnNumber, lineNumber and fileName

Do they serve a purpose?

Lol I had missed QuickJS has those. I see they code is very intentional, but there doesn't seem to be a good reason for them to be exposed to JS I guess? @bnoordhuis thoughts?

[p-bakker]

Discovered those through SES lockdown, as it logs and removes anything that is non-standard (and the SES code already had quite a few 'exceptions' paths in their codebase to remove stuff the initial quickjs implementation has.

After commenting out the quickjs-related exceptions in the SEs codebase, I found the non-standard properties remaining in quickJS-NG were the ones mentioned above

[saghul]

👍

[saghul]

#716