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

Side by Side Diff: pkg/path/lib/src/style/posix.dart

Issue 309803003: Avoid using RegExps in path to work around issue 19090. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: code review Created 6 years, 6 months 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « pkg/path/lib/src/parsed_path.dart ('k') | pkg/path/lib/src/style/url.dart » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 library path.style.posix; 5 library path.style.posix;
6 6
7 import '../characters.dart' as chars;
7 import '../parsed_path.dart'; 8 import '../parsed_path.dart';
8 import '../internal_style.dart'; 9 import '../internal_style.dart';
9 10
10 /// The style for POSIX paths. 11 /// The style for POSIX paths.
11 class PosixStyle extends InternalStyle { 12 class PosixStyle extends InternalStyle {
12 PosixStyle(); 13 PosixStyle();
13 14
14 final name = 'posix'; 15 final name = 'posix';
15 final separator = '/'; 16 final separator = '/';
17 final separators = const ['/'];
18
19 // Deprecated properties.
20
16 final separatorPattern = new RegExp(r'/'); 21 final separatorPattern = new RegExp(r'/');
17 final needsSeparatorPattern = new RegExp(r'[^/]$'); 22 final needsSeparatorPattern = new RegExp(r'[^/]$');
18 final rootPattern = new RegExp(r'^/'); 23 final rootPattern = new RegExp(r'^/');
24 final relativeRootPattern = null;
25
26 bool containsSeparator(String path) => path.contains('/');
27
28 bool isSeparator(int codeUnit) => codeUnit == chars.SLASH;
29
30 bool needsSeparator(String path) =>
31 path.isNotEmpty && !isSeparator(path.codeUnitAt(path.length - 1));
32
33 String getRoot(String path) {
34 if (path.isNotEmpty && isSeparator(path.codeUnitAt(0))) return '/';
35 return null;
36 }
37
38 String getRelativeRoot(String path) => null;
19 39
20 String pathFromUri(Uri uri) { 40 String pathFromUri(Uri uri) {
21 if (uri.scheme == '' || uri.scheme == 'file') { 41 if (uri.scheme == '' || uri.scheme == 'file') {
22 return Uri.decodeComponent(uri.path); 42 return Uri.decodeComponent(uri.path);
23 } 43 }
24 throw new ArgumentError("Uri $uri must have scheme 'file:'."); 44 throw new ArgumentError("Uri $uri must have scheme 'file:'.");
25 } 45 }
26 46
27 Uri absolutePathToUri(String path) { 47 Uri absolutePathToUri(String path) {
28 var parsed = new ParsedPath.parse(path, this); 48 var parsed = new ParsedPath.parse(path, this);
29 if (parsed.parts.isEmpty) { 49 if (parsed.parts.isEmpty) {
30 // If the path is a bare root (e.g. "/"), [components] will 50 // If the path is a bare root (e.g. "/"), [components] will
31 // currently be empty. We add two empty components so the URL constructor 51 // currently be empty. We add two empty components so the URL constructor
32 // produces "file:///", with a trailing slash. 52 // produces "file:///", with a trailing slash.
33 parsed.parts.addAll(["", ""]); 53 parsed.parts.addAll(["", ""]);
34 } else if (parsed.hasTrailingSeparator) { 54 } else if (parsed.hasTrailingSeparator) {
35 // If the path has a trailing slash, add a single empty component so the 55 // If the path has a trailing slash, add a single empty component so the
36 // URI has a trailing slash as well. 56 // URI has a trailing slash as well.
37 parsed.parts.add(""); 57 parsed.parts.add("");
38 } 58 }
39 59
40 return new Uri(scheme: 'file', pathSegments: parsed.parts); 60 return new Uri(scheme: 'file', pathSegments: parsed.parts);
41 } 61 }
42 } 62 }
OLDNEW
« no previous file with comments | « pkg/path/lib/src/parsed_path.dart ('k') | pkg/path/lib/src/style/url.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698