Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(417)

Issue 39148: In List::Add, correctly handle the case of adding a reference to a... (Closed)

Created:
11 years, 9 months ago by Kevin Millikin (Chromium)
Modified:
9 years, 7 months ago
Reviewers:
Kasper Lund
CC:
v8-dev
Visibility:
Public.

Description

In List::Add, correctly handle the case of adding a reference to a preexisting list element to a list, and to not return anything (we never used the return value). Remove List::Insert, it is not currently used or needed. Change List::AddBlock to take a copy of the element to be replicated rather than a reference. Committed: http://code.google.com/p/v8/source/detail?r=1424

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -27 lines) Patch
M src/list.h View 2 chunks +5 lines, -9 lines 0 comments Download
M src/list-inl.h View 1 chunk +9 lines, -18 lines 1 comment Download

Messages

Total messages: 2 (0 generated)
Kevin Millikin (Chromium)
11 years, 9 months ago (2009-03-04 20:00:18 UTC) #1
Kasper Lund
11 years, 9 months ago (2009-03-05 06:21:03 UTC) #2
LGTM.

http://codereview.chromium.org/39148/diff/1/3
File src/list-inl.h (right):

http://codereview.chromium.org/39148/diff/1/3#newcode46
Line 46: // Since the element reference could be an element of the list,
It would be nice with a test of this functionality.

Powered by Google App Engine
This is Rietveld 408576698