Subtask of bug 14868.
Created attachment 16931 [details]
Comment on attachment 16931 [details]
+JSValue* LocalVarDeleteNode::evaluate(ExecState* exec)
Unused parameter. Will warn when you merge with my settings.
+ , ident(ident)
+ , args(args.release())
These should both pass PlacementNewAdopt. See my recent patch for details.
Created attachment 16932 [details]
Factored ASSERT code out into canSkipLookup. Removed unused parameter.
Also used the right initializers.
Comment on attachment 16932 [details]
I'm worried about warnings in release builds due to the unused function.
Committed revision 27210, with an #if !ASSERT_DISABLED.