RESOLVED FIXED 233341
SubSpace constructors should take a const HeapCellType& instead of a HeapCellType*. 
https://bugs.webkit.org/show_bug.cgi?id=233341
Summary SubSpace constructors should take a const HeapCellType& instead of a HeapCel...
Mark Lam
Reported 2021-11-18 16:22:22 PST
This helps document that the HeapCellTypes are immutable once they are instantiated, and that SubSpaces won't be modifying them. Also remove the const on CellAttributes return values since it is not needed nor meaningful.
Attachments
[fast-cq] proposed patch. (197.57 KB, patch)
2021-11-18 16:30 PST, Mark Lam 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Radar WebKit Bug Importer
Comment 1 2021-11-18 16:22:56 PST
<rdar://problem/85573277>
Mark Lam
Comment 2 2021-11-18 16:30:33 PST
Created attachment 444756 [details] [fast-cq] proposed patch.
Yusuke Suzuki
Comment 3 2021-11-18 17:45:49 PST
Comment on attachment 444756 [details] [fast-cq] proposed patch. r=me
Mark Lam
Comment 4 2021-11-18 19:01:36 PST
Comment on attachment 444756 [details] [fast-cq] proposed patch. Thanks for the review.
EWS
Comment 5 2021-11-18 19:03:56 PST
Committed r286042 (244430@main): <https://commits.webkit.org/244430@main> All reviewed patches have been landed. Closing bug and clearing flags on attachment 444756 [details].
Note You need to log in before you can comment on or make changes to this bug.