| Summary: | Add copy constructor and assignment operator to Ref<> | ||||||
|---|---|---|---|---|---|---|---|
| Product: | WebKit | Reporter: | Darin Adler <darin> | ||||
| Component: | Web Template Framework | Assignee: | Darin Adler <darin> | ||||
| Status: | RESOLVED FIXED | ||||||
| Severity: | Normal | CC: | benjamin, cdumez, cmarcelo, dbates, esprehn+autocc, ews-watchlist, ggaren, kangil.han, kling, sam, webkit-bug-importer | ||||
| Priority: | P2 | Keywords: | InRadar | ||||
| Version: | WebKit Nightly Build | ||||||
| Hardware: | Unspecified | ||||||
| OS: | Unspecified | ||||||
| Attachments: |
|
||||||
|
Description
Darin Adler
2020-05-10 16:36:42 PDT
Created attachment 398986 [details]
Patch
Comment on attachment 398986 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=398986&action=review > Source/WTF/ChangeLog:19 > + These follow a similar patter to the move constructor and assignment operator. probably should spell it "pattern". I am glad Ref will finally have this. Committed r261467: <https://trac.webkit.org/changeset/261467> Great change! :) In retrospect, efforts to avoid refcount churn are probably best limited to situations where it has a measurable impact on performance, and should not be a goal in itself. Thank you, sir. Great to get a vote of confidence from the "founder of the [Ref class template] feast". |