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

Issue 11418010: Do not use StringBuffer to create strings from individual char (Closed)

Created:
8 years, 1 month ago by Mads Ager (google)
Modified:
8 years, 1 month ago
Reviewers:
Søren Gjesse
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Do not use StringBuffer to create strings from individual char codes in the HTTP parser. This was simple to do but we really should fix the performance of the string buffer for this case. R=sgjesse@google.com BUG= Committed: https://code.google.com/p/dart/source/detail?r=14967

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+26 lines, -26 lines) Patch
M sdk/lib/io/http_parser.dart View 16 chunks +26 lines, -26 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Mads Ager (google)
8 years, 1 month ago (2012-11-15 15:23:20 UTC) #1
Søren Gjesse
8 years, 1 month ago (2012-11-15 15:24:06 UTC) #2
LGTM!

Powered by Google App Engine
This is Rietveld 408576698