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

Issue 343583002: Make the internal parts of our external API a bit more const-correct. (Closed)

Created:
6 years, 6 months ago by Sven Panne
Modified:
6 years, 6 months ago
Reviewers:
Michael Starzinger
CC:
v8-dev, Paweł Hajdan Jr.
Visibility:
Public.

Description

Make the internal parts of our external API a bit more const-correct. R=mstarzinger@chromium.org Committed: https://code.google.com/p/v8/source/detail?r=21891

Patch Set 1 #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+18 lines, -16 lines) Patch
M include/v8.h View 6 chunks +18 lines, -16 lines 1 comment Download

Messages

Total messages: 3 (0 generated)
Sven Panne
More to come... :-}
6 years, 6 months ago (2014-06-18 13:22:06 UTC) #1
Michael Starzinger
LGTM. https://codereview.chromium.org/343583002/diff/1/include/v8.h File include/v8.h (right): https://codereview.chromium.org/343583002/diff/1/include/v8.h#newcode5638 include/v8.h:5638: kIsolateEmbedderDataOffset + slot * kApiPointerSize; nit: There are ...
6 years, 6 months ago (2014-06-18 13:48:16 UTC) #2
Sven Panne
6 years, 6 months ago (2014-06-20 07:20:57 UTC) #3
Message was sent while issue was closed.
Committed patchset #1 manually as r21891 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698