WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
247694
Remove unnecessary semicolons in various files
https://bugs.webkit.org/show_bug.cgi?id=247694
Summary
Remove unnecessary semicolons in various files
Ahmad Saleem
Reported
2022-11-09 13:28:55 PST
Hi Team, This is just to remove trialing ; on few of files within Webkit and it is just to create bug so can land PR later.
https://github.com/WebKit/WebKit/blob/d7bab9d6f7c45b99da17d78a464eac1fedff0ead/Source/WebCore/css/SelectorChecker.cpp#L396
https://github.com/WebKit/WebKit/blob/5245130a89305c2bfa17d07f6f843e1073ccac6e/Source/WebCore/platform/ScrollView.h#L117
https://github.com/WebKit/WebKit/blob/598cda65c4a2c2af24667bc86fdc1ed5cbafec23/Source/JavaScriptCore/parser/Parser.cpp#L1437
https://github.com/WebKit/WebKit/blob/598cda65c4a2c2af24667bc86fdc1ed5cbafec23/Source/JavaScriptCore/parser/Parser.cpp#L2398
https://github.com/WebKit/WebKit/blob/598cda65c4a2c2af24667bc86fdc1ed5cbafec23/Source/JavaScriptCore/parser/Parser.cpp#L2667
https://github.com/WebKit/WebKit/blob/598cda65c4a2c2af24667bc86fdc1ed5cbafec23/Source/WebCore/workers/WorkerMessagingProxy.cpp#L142
https://github.com/WebKit/WebKit/blob/598cda65c4a2c2af24667bc86fdc1ed5cbafec23/Source/WebCore/workers/service/context/ServiceWorkerThread.cpp#L101
https://github.com/WebKit/WebKit/blob/598cda65c4a2c2af24667bc86fdc1ed5cbafec23/Source/WebCore/workers/shared/context/SharedWorkerThreadProxy.h#L93
and any other. Thanks!
Attachments
Add attachment
proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1
2022-11-16 13:29:22 PST
<
rdar://problem/102434342
>
Andrew Fryer
Comment 2
2024-07-09 11:48:40 PDT
Pull request:
https://github.com/WebKit/WebKit/pull/30608
EWS
Comment 3
2024-07-11 15:16:50 PDT
Committed
280878@main
(b4dbf7a9e460): <
https://commits.webkit.org/280878@main
> Reviewed commits have been landed. Closing PR #30608 and removing active labels.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug