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

Unified Diff: utils/pub/path.dart

Issue 11557008: Make pub publish more user friendly: (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Merge in path changes. Created 8 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « utils/pub/io.dart ('k') | utils/pub/validator.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: utils/pub/path.dart
diff --git a/utils/pub/path.dart b/utils/pub/path.dart
index bcfbf7ce1b93b719db785343c4d675c869cda022..593396d0b83490de1be419a829c36ac0b09b715a 100644
--- a/utils/pub/path.dart
+++ b/utils/pub/path.dart
@@ -102,10 +102,13 @@ String join(String part1, [String part2, String part3, String part4,
// TODO(nweiz): add a UNC example for Windows once issue 7323 is fixed.
/// Splits [path] into its components using the current platform's [separator].
-/// Example:
///
/// path.split('path/to/foo'); // -> ['path', 'to', 'foo']
///
+/// The path will *not* be normalized before splitting.
+///
+/// path.split('path/../foo'); // -> ['path', '..', 'foo']
+///
/// If [path] is absolute, the root directory will be the first element in the
/// array. Example:
///
@@ -315,6 +318,10 @@ class Builder {
///
/// builder.split('path/to/foo'); // -> ['path', 'to', 'foo']
///
+ /// The path will *not* be normalized before splitting.
+ ///
+ /// builder.split('path/../foo'); // -> ['path', '..', 'foo']
+ ///
/// If [path] is absolute, the root directory will be the first element in the
/// array. Example:
///
« no previous file with comments | « utils/pub/io.dart ('k') | utils/pub/validator.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698