Issue 6765026: [Arguments] Fix a small bug and change some test expectations.

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Issue 6765026: [Arguments] Fix a small bug and change some test expectations. (Closed)

Created:
9 years, 8 months ago by Kevin Millikin (Chromium)

Modified:
9 years, 7 months ago

Reviewers:
Mads Ager (chromium)

CC:
v8-dev

Base URL:
https://v8.googlecode.com/svn/branches/experimental/arguments

Visibility:
Public.

Description

[Arguments] Fix a small bug and change some test expectations. When handling duplicate parameter names, the first duplicate named parameter "won" the prize of being aliased by the arguments object. It should be the last duplicate that wins. Also, remove the FAIL expectation for some tests that we now pass. R=ager@chromium.org Committed: http://code.google.com/p/v8/source/detail?r=7441

Patch Set 1 #

Created: 9 years, 8 months ago

Download [raw] [tar.bz2]

	Unified diffs	Side-by-side diffs	Stats (+2 lines, -15 lines)			Patch
M	src/runtime.cc	View	1 chunk	+2 lines, -2 lines	0 comments	Download
M	test/mozilla/mozilla.status	View	1 chunk	+0 lines, -5 lines	0 comments	Download
M	test/sputnik/sputnik.status	View	1 chunk	+0 lines, -8 lines	0 comments	Download

Messages

Total messages: 2 (0 generated)

Expand Messages | Collapse Messages

LGTM

Expand Messages | Collapse Messages