RESOLVED FIXED 145776
Access GCController instance via GCController::singleton() instead of a free function
https://bugs.webkit.org/show_bug.cgi?id=145776
Summary Access GCController instance via GCController::singleton() instead of a free ...
Chris Dumez
Reported 2015-06-08 16:12:26 PDT
Access GCController instance via GCController::singleton() instead of a free function as per coding style and for consistency with other singleton classes in the codebase.
Attachments
Patch (18.12 KB, patch)
2015-06-08 16:18 PDT, Chris Dumez
darin: review+
Chris Dumez
Comment 1 2015-06-08 16:18:11 PDT
Darin Adler
Comment 2 2015-06-08 17:26:46 PDT
Comment on attachment 254521 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=254521&action=review > Source/WebCore/bindings/js/GCController.cpp:50 > + static NeverDestroyed<GCController> gcController; > + return gcController; I would have just named this local "controller". > Source/WebCore/bindings/js/GCController.h:37 > + WTF_MAKE_NONCOPYABLE(GCController); WTF_MAKE_FAST_ALLOCATED; No need for WTF_MAKE_FAST_ALLOCATED.
Chris Dumez
Comment 3 2015-06-08 17:28:49 PDT
Chris Dumez
Comment 4 2015-06-08 17:30:02 PDT
Comment on attachment 254521 [details] Patch Made nit fixes before landing.
Note You need to log in before you can comment on or make changes to this bug.