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

Issue 6588104: Merge 76320 - Implementation of an STL compatible allocator for Courgette on ... (Closed)

Created:
9 years, 9 months ago by tommi (sloooow) - chröme
Modified:
9 years, 7 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

Merge 76320 - Implementation of an STL compatible allocator for Courgette on Windows. This is to better handle low memory situations when applying a differential patch to a large Chrome setup. For reading input files, I'm also switching to using memory mapped files instead of ReadFileToString to reduce the load on the heap. TEST=courgette.exe should succeed in applying a patch between two chrome 10.x archives on an XP machine with 180MB of physical memory and no page file. BUG=72459, 73209 Review URL: http://codereview.chromium.org/6597038 TBR=tommi@chromium.org,kerz@chromium.org Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=76471

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+63 lines, -62 lines) Patch
M courgette/adjustment_method.cc View 1 chunk +1 line, -1 line 0 comments Download
M courgette/adjustment_method_2.cc View 1 chunk +1 line, -1 line 0 comments Download
M courgette/assembly_program.h View 4 chunks +6 lines, -3 lines 0 comments Download
M courgette/courgette.gyp View 1 chunk +2 lines, -0 lines 0 comments Download
M courgette/encoded_program.h View 3 chunks +18 lines, -12 lines 0 comments Download
M courgette/encoded_program.cc View 12 chunks +23 lines, -20 lines 0 comments Download
M courgette/ensemble_apply.cc View 1 chunk +8 lines, -26 lines 0 comments Download
A + courgette/memory_allocator.h View 0 chunks +-1 lines, --1 lines 0 comments Download
A + courgette/memory_allocator.cc View 0 chunks +-1 lines, --1 lines 0 comments Download
M courgette/streams.h View 2 chunks +6 lines, -1 line 0 comments Download

Messages

Total messages: 1 (0 generated)
tommi (sloooow) - chröme
9 years, 9 months ago (2011-03-02 00:45:45 UTC) #1

          

Powered by Google App Engine
This is Rietveld 408576698