RESOLVED FIXED 216225
Make CompactUniquePtrTuple actually work with subclassing and custom deleter
https://bugs.webkit.org/show_bug.cgi?id=216225
Summary Make CompactUniquePtrTuple actually work with subclassing and custom deleter
Ryosuke Niwa
Reported 2020-09-06 02:26:21 PDT
There are some bugs :(
Attachments
Fixes the bugs (13.18 KB, patch)
2020-09-06 02:33 PDT, Ryosuke Niwa
darin: review+
Ryosuke Niwa
Comment 1 2020-09-06 02:33:31 PDT
Created attachment 408121 [details] Fixes the bugs
Darin Adler
Comment 2 2020-09-06 14:09:36 PDT
Comment on attachment 408121 [details] Fixes the bugs View in context: https://bugs.webkit.org/attachment.cgi?id=408121&action=review > Source/WTF/wtf/CompactUniquePtrTuple.h:69 > + CompactUniquePtrTuple<T, Type, Deleter> moved { WTFMove(other) }; Here inside a member we should not need to repeat the template arguments, so although it can’t use auto, it can be like this: CompactUniquePtrTuple moved { WTFMove(other) }; > Source/WTF/wtf/CompactUniquePtrTuple.h:118 > + template <typename U, typename Y, typename D> friend class CompactUniquePtrTuple; No need for the U/Y/D here. Can just be "typename, typename, typename".
Ryosuke Niwa
Comment 3 2020-09-07 20:46:29 PDT
Thanks for the reviews! (In reply to Darin Adler from comment #2) > Comment on attachment 408121 [details] > Fixes the bugs > > View in context: > https://bugs.webkit.org/attachment.cgi?id=408121&action=review > > > Source/WTF/wtf/CompactUniquePtrTuple.h:69 > > + CompactUniquePtrTuple<T, Type, Deleter> moved { WTFMove(other) }; > > Here inside a member we should not need to repeat the template arguments, so > although it can’t use auto, it can be like this: Done. > CompactUniquePtrTuple moved { WTFMove(other) }; > > > Source/WTF/wtf/CompactUniquePtrTuple.h:118 > > + template <typename U, typename Y, typename D> friend class CompactUniquePtrTuple; > > No need for the U/Y/D here. Can just be "typename, typename, typename". Removed.
Ryosuke Niwa
Comment 4 2020-09-07 20:50:16 PDT
Radar WebKit Bug Importer
Comment 5 2020-09-07 20:51:17 PDT
Note You need to log in before you can comment on or make changes to this bug.