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

Issue 10383103: Split the URI library across multiple files. (Closed)

Created:
8 years, 7 months ago by ahe
Modified:
8 years, 7 months ago
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Split the URI library across multiple files. Committed: https://code.google.com/p/dart/source/detail?r=7543

Patch Set 1 #

Total comments: 4

Patch Set 2 : Created helper.dart with SVN #

Unified diffs Side-by-side diffs Delta from patch set Stats (+64 lines, -26 lines) Patch
M dart/lib/uri/uri.dart View 2 chunks +2 lines, -24 lines 0 comments Download
M dart/runtime/bin/bin.gypi View 2 chunks +21 lines, -2 lines 0 comments Download
M dart/runtime/bin/uri_sources.gypi View 1 chunk +1 line, -0 lines 0 comments Download
A dart/runtime/tools/concat_library.py View 1 chunk +40 lines, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
ahe
https://chromiumcodereview.appspot.com/10383103/diff/1/dart/lib/uri/uri.dart File dart/lib/uri/uri.dart (right): https://chromiumcodereview.appspot.com/10383103/diff/1/dart/lib/uri/uri.dart#newcode7 dart/lib/uri/uri.dart:7: #source('helpers.dart'); Note: there is no pressing need to add ...
8 years, 7 months ago (2012-05-10 11:42:59 UTC) #1
ahe
8 years, 7 months ago (2012-05-10 11:45:55 UTC) #2
ngeoffray
LGTM https://chromiumcodereview.appspot.com/10383103/diff/1/dart/lib/uri/helpers.dart File dart/lib/uri/helpers.dart (right): https://chromiumcodereview.appspot.com/10383103/diff/1/dart/lib/uri/helpers.dart#newcode19 dart/lib/uri/helpers.dart:19: } else if ("." == segment) { consistency: ...
8 years, 7 months ago (2012-05-11 12:41:23 UTC) #3
ahe
Thank you, Nicolas. https://chromiumcodereview.appspot.com/10383103/diff/1/dart/lib/uri/helpers.dart File dart/lib/uri/helpers.dart (right): https://chromiumcodereview.appspot.com/10383103/diff/1/dart/lib/uri/helpers.dart#newcode19 dart/lib/uri/helpers.dart:19: } else if ("." == segment) ...
8 years, 7 months ago (2012-05-11 12:51:32 UTC) #4
Siggi Cherem (dart-lang)
8 years, 7 months ago (2012-05-12 00:33:48 UTC) #5
lgtm

Powered by Google App Engine
This is Rietveld 408576698