Bug 251346

Summary: Unnecessarily using std::variant and std::monostate when we could just use nullptr instead
Product: WebKit Reporter: Darin Adler <darin>
Component: Web InspectorAssignee: Darin Adler <darin>
Status: RESOLVED FIXED    
Severity: Normal CC: dino, inspector-bugzilla-changes, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: WebKit Nightly Build   
Hardware: All   
OS: All   

Darin Adler
Reported 2023-01-29 22:18:17 PST
Noticed that some code is using std::variant and std::monostate for a case where nullptr will do the trick.
Attachments
Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2023-01-29 22:19:09 PST
<rdar://problem/104809022>
Darin Adler
Comment 2 2023-01-29 23:37:36 PST
Pull request: https://github.com/WebKit/WebKit/pull/9302
EWS
Comment 3 2023-01-30 11:01:02 PST
Committed 259572@main (37f282e65d2e): <https://commits.webkit.org/259572@main> Reviewed commits have been landed. Closing PR #9302 and removing active labels.
Note You need to log in before you can comment on or make changes to this bug.