Bug 124166

Summary: setupArguments needs 5 argument overload
Product: WebKit Reporter: Alex Christensen <alex.christensen>
Component: JavaScriptCoreAssignee: Nobody <webkit-unassigned>
Status: RESOLVED WONTFIX    
Severity: Normal CC: buildbot, eflews.bot, ggaren, gyuyoung.kim, msaboff, philn, rniwa, xan.lopez
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: PC   
OS: Windows 8   
Attachments:
Description Flags
Patch
none
Patch none

Alex Christensen
Reported 2013-11-11 14:00:16 PST
Compiling line 953 of Repatch.cpp fails on Win64 because there is no overload of setupArguments with 5 arguments. I can't test it because the Win64 JIT is dead, but this makes it compile successfully.
Attachments
Patch (1.49 KB, patch)
2013-11-11 14:01 PST, Alex Christensen 		no flags
DetailsFormatted DiffDiff
Patch (1.54 KB, patch)
2013-11-11 14:16 PST, Alex Christensen 		no flags
DetailsFormatted DiffDiff
Show Obsolete (1) View All Add attachment proposed patch, testcase, etc.
Alex Christensen
Comment 1 2013-11-11 14:01:37 PST
Created attachment 216608 [details] Patch
EFL EWS Bot
Comment 2 2013-11-11 14:05:36 PST
Comment on attachment 216608 [details] Patch Attachment 216608 [details] did not pass efl-ews (efl): Output: http://webkit-queues.appspot.com/results/22539452
Build Bot
Comment 3 2013-11-11 14:08:33 PST
Comment on attachment 216608 [details] Patch Attachment 216608 [details] did not pass mac-ews (mac): Output: http://webkit-queues.appspot.com/results/21249505
Alex Christensen
Comment 4 2013-11-11 14:16:56 PST
Created attachment 216612 [details] Patch
Alex Christensen
Comment 5 2013-12-16 19:44:55 PST
This is no longer a problem
Note You need to log in before you can comment on or make changes to this bug.