Summary: | [Clang 10] Fix warning: definition of implicit copy assignment operator for 'PageBlock' is deprecated because it has a user-declared copy constructor | ||||||
---|---|---|---|---|---|---|---|
Product: | WebKit | Reporter: | Charlie Turner <cturner> | ||||
Component: | WebKitGTK | Assignee: | Charlie Turner <cturner> | ||||
Status: | RESOLVED FIXED | ||||||
Severity: | Normal | CC: | aperez, benjamin, bugs-noreply, cdumez, cmarcelo, darin, ews-watchlist | ||||
Priority: | P2 | ||||||
Version: | WebKit Nightly Build | ||||||
Hardware: | Unspecified | ||||||
OS: | Unspecified | ||||||
Attachments: |
|
Description
Charlie Turner
2020-04-20 08:22:39 PDT
Created attachment 396990 [details]
Patch
Patch LGTM, I am tempted to r+ given that all the EWS builders are green, but would be more confident if someone else would also take a quick look as this is fairly low-level code 🤔️ Comment on attachment 396990 [details]
Patch
Let's wait until tomorrow to land this in case someone else wants to take a
look, but the more I stare at it, the more I am convinced it's all right :D
Looks right to me. Committed r260431: <https://trac.webkit.org/changeset/260431> All reviewed patches have been landed. Closing bug and clearing flags on attachment 396990 [details]. |