Add *explicit* keyword to WebCore/rendering to avoid implicit type conversion.
Created attachment 183621 [details] Patch
Comment on attachment 183621 [details] Patch This is OK, but the patch has minimal benefit. There is near zero risk of accidentally doing implicit construction of these classes from these pointer types. Just no real chance this will arise idiomatically. Except maybe RenderArena.
Comment on attachment 183621 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=183621&action=review > Source/WebCore/rendering/RenderGeometryMap.h:46 > - RenderGeometryMapStep(const RenderGeometryMapStep& o) > + explicit RenderGeometryMapStep(const RenderGeometryMapStep& o) Oops, missed this one. This is wrong. Please don’t land this change. There is no reason to mark a copy constructor explicit!
Created attachment 183660 [details] Patch
(In reply to comment #3) > (From update of attachment 183621 [details]) > View in context: https://bugs.webkit.org/attachment.cgi?id=183621&action=review > > > Source/WebCore/rendering/RenderGeometryMap.h:46 > > - RenderGeometryMapStep(const RenderGeometryMapStep& o) > > + explicit RenderGeometryMapStep(const RenderGeometryMapStep& o) > > Oops, missed this one. This is wrong. Please don’t land this change. There is no reason to mark a copy constructor explicit! Thank you for your review. There was wrong one as you pointed out. Fixed it.
Comment on attachment 183660 [details] Patch Clearing flags on attachment: 183660 Committed r140291: <http://trac.webkit.org/changeset/140291>
All reviewed patches have been landed. Closing bug.