In jsc.cpp, there is currently checkSyntax() and checkModuleSyntax(). The former expects and filename, while the latter expects a string of javascript code. This would introduce a function called checkScriptSyntax() that performed the same operation as checkModuleSyntax(), without the module specific semantics.
Created attachment 439153 [details] Patch
Comment on attachment 439153 [details] Patch r=me
? found in /Volumes/Data/worker/Commit-Queue/build/JSTests/ChangeLog does not appear to be a valid reviewer according to contributors.json. /Volumes/Data/worker/Commit-Queue/build/JSTests/ChangeLog neither lists a valid reviewer nor contains the string "Unreviewed" or "Rubber stamp" (case insensitive).
Comment on attachment 439153 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=439153&action=review Setting r- since ChangeLog file is not correct. > Source/JavaScriptCore/ChangeLog:6 > + Reviewed by ?. Default line is necessary for commit-queue. https://webkit.org/contributing-code/#changelog-files You can check the instructions of contribution in https://webkit.org/contributing-code/ :)
I didn't know if I should put a name before I got the actual review ;)
Created attachment 439176 [details] Patch
Created attachment 439178 [details] Patch
Committed r283062 (242120@main): <https://commits.webkit.org/242120@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 439178 [details].
<rdar://problem/83513547>