Bug 228808

Summary: Change WebSocket.cpp to @no-unify, since it’s hard to do that for someone without a Mac
Product: WebKit Reporter: Darin Adler <darin>
Component: WebCore Misc.Assignee: Darin Adler <darin>
Status: RESOLVED FIXED    
Severity: Normal CC: andersca, mcatanzaro, webkit-bug-importer
Priority: P2 Keywords: InRadar
Version: Other   
Hardware: All   
OS: All   
Bug Depends on:    
Bug Blocks: 226557    
Attachments:
Description Flags
Patch none

Description Darin Adler 2021-08-04 18:15:51 PDT
Michael Catanzaro asked for help making WebSocket.cpp a @no-unify file, including the Xcode project changes.
Comment 1 Darin Adler 2021-08-04 18:18:26 PDT
Created attachment 434960 [details]
Patch
Comment 2 Darin Adler 2021-08-04 18:20:31 PDT
Not thrilled to be adding more @no-unify. We’d like to do the reverse and have fewer separately compiled files!
Comment 3 EWS 2021-08-04 19:52:07 PDT
Committed r280677 (240279@main): <https://commits.webkit.org/240279@main>

All reviewed patches have been landed. Closing bug and clearing flags on attachment 434960 [details].
Comment 4 Radar WebKit Bug Importer 2021-08-04 19:53:18 PDT
<rdar://problem/81546248>
Comment 5 Michael Catanzaro 2021-08-05 06:43:07 PDT
Thanks Darin!

(In reply to Darin Adler from comment #2)
> Not thrilled to be adding more @no-unify. We’d like to do the reverse and
> have fewer separately compiled files!

I know. :/ I've tried my best in bug #226557 and concluded it's either do this, or build all of WebCore with -Wno-stringop-overread. I'd rather suppress it for this one file than the entire project.