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

Issue 477513002: Fix analyzer warnings in http_multi_server. (Closed)

Created:
6 years, 4 months ago by nweiz
Modified:
6 years, 4 months ago
Reviewers:
Bob Nystrom
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Fix analyzer warnings in http_multi_server. R=rnystrom@google.com TBR Committed: https://code.google.com/p/dart/source/detail?r=39236

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+15 lines, -3 lines) Patch
M pkg/http_multi_server/CHANGELOG.md View 1 chunk +6 lines, -0 lines 0 comments Download
M pkg/http_multi_server/lib/src/multi_headers.dart View 2 chunks +8 lines, -2 lines 0 comments Download
M pkg/http_multi_server/pubspec.yaml View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3 (0 generated)
nweiz
6 years, 4 months ago (2014-08-14 01:25:16 UTC) #1
nweiz
Committed patchset #1 manually as 39236 (presubmit successful).
6 years, 4 months ago (2014-08-14 01:25:34 UTC) #2
Bob Nystrom
6 years, 4 months ago (2014-08-14 16:47:46 UTC) #3
Message was sent while issue was closed.
LGTM. I should have caught the remove() ones. :(

Powered by Google App Engine
This is Rietveld 408576698