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

Issue 7869: Apply Daniel's patch for array index strings. (Closed)

Created:
12 years, 2 months ago by Kasper Lund
Modified:
9 years, 7 months ago
Reviewers:
kodandersson, bak
CC:
v8-dev
Visibility:
Public.

Description

Apply Daniel's patch for array index strings. Committed: http://code.google.com/p/v8/source/detail?r=560

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+41 lines, -14 lines) Patch
M AUTHORS View 1 chunk +1 line, -0 lines 0 comments Download
M src/ic-ia32.cc View 3 chunks +15 lines, -4 lines 0 comments Download
M src/objects.h View 3 chunks +5 lines, -2 lines 0 comments Download
M src/objects.cc View 3 chunks +17 lines, -6 lines 1 comment Download
M src/objects-inl.h View 1 chunk +3 lines, -2 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Kasper Lund
12 years, 2 months ago (2008-10-22 12:53:07 UTC) #1
bak
LGTM, Lars http://codereview.chromium.org/7869/diff/1/4 File src/objects.cc (right): http://codereview.chromium.org/7869/diff/1/4#newcode3922 Line 3922: Hash(); // force computation of hash ...
12 years, 2 months ago (2008-10-22 13:25:14 UTC) #2
kodandersson
12 years, 2 months ago (2008-10-22 14:47:26 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld 408576698