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

Issue 46673004: Remove unused ValueInfo struct (Closed)

Created:
7 years, 1 month ago by Adrian Perez de Castro
Modified:
7 years, 1 month ago
Reviewers:
Benedikt Meurer
CC:
v8-dev
Visibility:
Public.

Description

Remove unused ValueInfo struct BUG= R=bmeurer@chromium.org Committed: https://code.google.com/p/v8/source/detail?r=17577

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -8 lines) Patch
M src/objects.h View 1 chunk +0 lines, -8 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
Adrian Perez de Castro
7 years, 1 month ago (2013-10-30 21:28:22 UTC) #1
Benedikt Meurer
Hey, Thanks for the patch. Before we can land it, you'll need to submit Committers ...
7 years, 1 month ago (2013-11-06 07:38:28 UTC) #2
Adrian Perez de Castro
On 2013/11/06 07:38:28, Benedikt Meurer wrote: > Hey, > > Thanks for the patch. Before ...
7 years, 1 month ago (2013-11-06 08:56:33 UTC) #3
Benedikt Meurer
On 2013/11/06 08:56:33, aperez wrote: > On 2013/11/06 07:38:28, Benedikt Meurer wrote: > > Hey, ...
7 years, 1 month ago (2013-11-08 07:36:23 UTC) #4
Benedikt Meurer
Committed patchset #1 manually as r17577 (presubmit successful).
7 years, 1 month ago (2013-11-08 07:45:21 UTC) #5
Adrian Perez de Castro
7 years, 1 month ago (2013-11-08 10:11:01 UTC) #6
Message was sent while issue was closed.
On 2013/11/08 07:36:23, Benedikt Meurer wrote:

> LGTM. I'll land it.

Thanks!

Powered by Google App Engine
This is Rietveld 408576698