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

Issue 26863002: Use C++ style raw (de)allocation in FrameDescription. (Closed)

Created:
7 years, 2 months ago by dusmil
Modified:
7 years, 2 months ago
CC:
v8-dev
Visibility:
Public.

Description

Use C++ style raw (de)allocation in FrameDescription. TEST= BUG=

Patch Set 1 #

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

Messages

Total messages: 5 (0 generated)
dusmil
7 years, 2 months ago (2013-10-10 15:54:42 UTC) #1
danno
I fundamentally don't have an issue with this CL, but I am curious what the ...
7 years, 2 months ago (2013-10-11 15:26:04 UTC) #2
dusmil
On 2013/10/11 15:26:04, danno wrote: > I fundamentally don't have an issue with this CL, ...
7 years, 2 months ago (2013-10-11 16:27:47 UTC) #3
danno
I don't think there's any utility to this change unless you make the entire source ...
7 years, 2 months ago (2013-10-14 17:21:15 UTC) #4
danno
7 years, 2 months ago (2013-10-14 17:21:17 UTC) #5
I don't think there's any utility to this change unless you make the entire
source base consistent, which I'm sure it's not. And even then, it's only a very
small cosmetic improvement in the grand scheme of things, so I'd prefer spending
time reviewing meatier changes.

Powered by Google App Engine
This is Rietveld 408576698