Chromium Code Reviews

Issue 9395073: Fix memory leak and missing #include in StartupDataDecompressor (Closed)

Created:
8 years, 10 months ago by Jakob Kummerow
Modified:
8 years, 10 months ago
Reviewers:
mnaganov (inactive)
CC:
v8-dev
Visibility:
Public.

Description

Fix memory leak and missing #include in StartupDataDecompressor BUG=v8:1960 TEST=compile with GYPFLAGS=-Dv8_compress_snapshot_data=bz2, then run d8 with valgrind Committed: https://code.google.com/p/v8/source/detail?r=10751

Patch Set 1 #

Unified diffs Side-by-side diffs Stats (+4 lines, -0 lines)
M src/api.cc View 2 chunks +4 lines, -0 lines 0 comments

Messages

Total messages: 2 (0 generated)
Jakob Kummerow
Please take a look.
8 years, 10 months ago (2012-02-20 10:43:40 UTC) #1
mnaganov (inactive)
8 years, 10 months ago (2012-02-20 10:50:53 UTC) #2
On 2012/02/20 10:43:40, Jakob wrote:
> Please take a look.

LGTM!

Thanks for fixing this!

Powered by Google App Engine