[Cocoa] Improve ARC compatibility of more code in JavaScriptCore
Created attachment 343440 [details] Patch
Created attachment 343977 [details] Patch
Created attachment 344027 [details] Patch
Committed r233409: <https://trac.webkit.org/changeset/233409>
<rdar://problem/41699383>
Comment on attachment 344027 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=344027&action=review > Source/JavaScriptCore/API/JSContextInternal.h:33 > - CallbackData *next; > + CallbackData* next; I thought in Objective-C++ code we followed the Objective-C style of putting the * next to the variable name rather than our C++ style?
(In reply to Keith Miller from comment #6) > Comment on attachment 344027 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=344027&action=review > > > Source/JavaScriptCore/API/JSContextInternal.h:33 > > - CallbackData *next; > > + CallbackData* next; > > I thought in Objective-C++ code we followed the Objective-C style of putting > the * next to the variable name rather than our C++ style? * placement depends on the type. For Objective-C classes, the * comes after the space.
Comment on attachment 344027 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=344027&action=review > Source/JavaScriptCore/API/JSVirtualMachine.mm:-190 > - [ownedObjects release]; This change should not have been included in the patch, and was a mismerge. It introduced a storage leak fixed in bug 201803.