RESOLVED FIXED206840
Python 3: generate-js-builtins hits SyntaxWarning for "is 0" 
https://bugs.webkit.org/show_bug.cgi?id=206840
Summary Python 3: generate-js-builtins hits SyntaxWarning for "is 0"
Stephan Szabo
Reported 2020-01-27 13:15:03 PST
After switching up to python 3.8, generate-js-builtins.py hits a SyntaxWarning for the use of "is 0". Apparently, per https://bugs.python.org/issue34850 this sort of usage happens to work on CPython by accident, but is not guaranteed to work on other python implementation or necessarily continue working in CPython. The warning suggests replacing is with == for such cases.
Attachments
Replace use of "is 0" with "== 0". (1.30 KB, patch)
2020-01-27 13:23 PST, Stephan Szabo 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Stephan Szabo
Comment 1 2020-01-27 13:23:26 PST
Created attachment 388892 [details] Replace use of "is 0" with "== 0".
EWS Watchlist
Comment 2 2020-01-27 13:24:09 PST
This patch modifies the JS builtins code generator. Please ensure that you have rebaselined any generator test results (i.e., by running `Tools/Scripts/run-builtins-generator-tests --reset-results`)
Blaze Burg
Comment 3 2020-01-27 13:46:24 PST
Comment on attachment 388892 [details] Replace use of "is 0" with "== 0". rs=me
WebKit Commit Bot
Comment 4 2020-01-27 15:23:05 PST
Comment on attachment 388892 [details] Replace use of "is 0" with "== 0". Clearing flags on attachment: 388892 Committed r255186: <https://trac.webkit.org/changeset/255186>
WebKit Commit Bot
Comment 5 2020-01-27 15:23:06 PST
All reviewed patches have been landed. Closing bug.
Radar WebKit Bug Importer
Comment 6 2020-01-27 15:24:14 PST
<rdar://problem/58936818>
Note You need to log in before you can comment on or make changes to this bug.