Bug 282674

Summary:	Enforce `enum class X : bool { No, Yes }` instead of `{ Yes, No }`
Product:	WebKit	Reporter:	Gerald Squelart <g_squelart>
Component:	Tools / Tests	Assignee:	Gerald Squelart <g_squelart>
Status:	RESOLVED FIXED
Severity:	Normal	CC:	webkit-bug-importer
Priority:	P2	Keywords:	InRadar
Version:	WebKit Nightly Build
Hardware:	Unspecified
OS:	Unspecified
Bug Depends on:	254322, 254358
Bug Blocks:

Gerald Squelart

Reported 2024-11-05 22:24:46 PST

As of this writing, there are at least 7 instances of `{ Yes, No }` since bug 254322 and bug 254358 landed. Ideally, there should be a `check-webkit-style` checker to catch this risky ordering now and in the future.

Attachments
Add attachment proposed patch, testcase, etc.

Radar WebKit Bug Importer

Comment 1 2024-11-05 22:25:07 PST

<rdar://problem/139335415>

Gerald Squelart

Comment 2 2024-11-10 00:17:32 PST

Pull request: https://github.com/WebKit/WebKit/pull/36457

EWS

Comment 3 2024-11-12 11:08:41 PST

Committed 286492@main (f2554a7bf3fa): <https://commits.webkit.org/286492@main> Reviewed commits have been landed. Closing PR #36457 and removing active labels.

Note You need to log in before you can comment on or make changes to this bug.