66548 2011-08-19 05:50:02 -0700 check-webkit-style should allow out RefPtr parameters 2011-10-04 19:26:27 -0700 1 1 1 Unclassified WebKit Tools / Tests 528+ (Nightly build) Unspecified Unspecified RESOLVED DUPLICATE 67797 P2 Normal --- 1 pfeldman webkit-unassigned abarth darin eric levin oldest_to_newest 453765 0 pfeldman 2011-08-19 05:50:02 -0700 Source/WebCore/inspector/InspectorDOMAgent.h:131: The parameter type should use PassRefPtr instead of RefPtr. [readability/pass_ptr] [5] 453894 1 eric 2011-08-19 10:10:47 -0700 http://trac.webkit.org/browser/trunk/Source/WebCore/inspector/InspectorDOMAgent.h#L131 453895 2 eric 2011-08-19 10:10:55 -0700 void getAttributes(ErrorString*, int nodeId, RefPtr<InspectorArray>* result); 453897 3 eric 2011-08-19 10:12:30 -0700 I think other parts of the code would write that as a reference instead of a pointer. Or they'd return a PassRefPtr from the method itself. Also, "get" is not normally used in method names, except where dictated by the DOM api. I'm not sure if there is a bug here. 453906 4 darin 2011-08-19 10:21:01 -0700 (In reply to comment #3) > I think other parts of the code would write that as a reference instead of a pointer. Or they'd return a PassRefPtr from the method itself. Yes. > Also, "get" is not normally used in method names, except where dictated by the DOM api. I disagree; get is correctly used here to indicate a function that uses an out argument rather than a function result. But as Eric says, in this case there is no obvious reason for use of an out argument for a single function result. Returning a PassRefPtr is the idiom for this sort of thing. 477889 5 pfeldman 2011-10-04 13:40:43 -0700 > But as Eric says, in this case there is no obvious reason for use of an out argument for a single function result. Returning a PassRefPtr is the idiom for this sort of thing. This is a part of the inspector protocol, methods on the agents are called from the generated dispatcher. Generator was treating RefPtr as the rest of the types, using raw pointers for all out parameters. We'll change it to: void getAttributes(ErrorString*, int nodeId, RefPtr&<InspectorArray> result); to address this warning under https://bugs.webkit.org/show_bug.cgi?id=69366. 478117 6 levin 2011-10-04 19:26:27 -0700 I missed this when I fixed it, so I'l just resolve this as a duplicate of the bug that I fixed for this. If you're still get style errors about this, either let me know or feel free to submit a fix. (It should be trivial especially when you see my previous fix.) *** This bug has been marked as a duplicate of bug 67797 ***