Bug 142851

Summary: WeakGCMap pruning callback doesn't need to be a separate function.
Product: WebKit Reporter: Andreas Kling <kling>
Component: JavaScriptCoreAssignee: Andreas Kling <kling>
Status: NEW    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Attachments:
Description Flags
Patch none

Andreas Kling
Reported 2015-03-18 17:35:08 PDT
Looking over WeakGCMap again, I realized that pruneStaleEntries() doesn't need to be a separate function, we can just fold it into the lambda we register with the heap.
Attachments
Patch (2.32 KB, patch)
2015-03-18 17:37 PDT, Andreas Kling 		no flags
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Andreas Kling
Comment 1 2015-03-18 17:37:26 PDT
Created attachment 248995 [details] Patch
Andreas Kling
Comment 2 2015-03-18 20:35:50 PDT
Comment on attachment 248995 [details] Patch This fails on Windows :( c:\cygwin\home\buildbot\webkit\source\javascriptcore\runtime\WeakGCMapInlines.h(39): error C2899: typename cannot be used outside a template declaration
Note You need to log in before you can comment on or make changes to this bug.