commit 917547487a3773527f8f5512fac6e21764a27493
author Natalie Weizenbaum <nweiz@google.com> Thu Jun 08 15:16:50 2017 -0700
committer GitHub <noreply@github.com> Thu Jun 08 15:16:50 2017 -0700
tree 095ed393cf560f90a90785d31e6afe330eabdd17
parent 0bb73f1ce388f780b3bc0d12c09c3ee2375539ed

Fix a normalization bug. (#26)

Closes #24

CHANGELOG.md

diff --git a/CHANGELOG.md b/CHANGELOG.md
index f7dde67..67c849c 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,3 +1,7 @@
+## 1.4.2
+
+* Normalize `c:\foo\.` to `c:\foo`.
+
 ## 1.4.1
 
 * Root-relative URLs like `/foo` are now resolved relative to the drive letter

lib/src/context.dart

diff --git a/lib/src/context.dart b/lib/src/context.dart
index c5ebb69..a00ca29 100644
--- a/lib/src/context.dart
+++ b/lib/src/context.dart
@@ -397,7 +397,7 @@
     // Single dots and double dots are normalized to directory traversals.
     if (previous == chars.PERIOD &&
         (previousPrevious == null ||
-         previousPrevious == chars.SLASH ||
+         style.isSeparator(previousPrevious) ||
          previousPrevious == chars.PERIOD)) {
       return true;
     }

pubspec.yaml

diff --git a/pubspec.yaml b/pubspec.yaml
index 48e799b..ff101d6 100644
--- a/pubspec.yaml
+++ b/pubspec.yaml
@@ -1,5 +1,5 @@
 name: path
-version: 1.4.1
+version: 1.4.2-dev
 author: Dart Team <misc@dartlang.org>
 description: >
  A string-based path manipulation library. All of the path operations you know

test/windows_test.dart

diff --git a/test/windows_test.dart b/test/windows_test.dart
index ad58774..364db34 100644
--- a/test/windows_test.dart
+++ b/test/windows_test.dart
@@ -330,6 +330,7 @@
     test('eliminates "." parts', () {
       expect(context.normalize(r'.\'), '.');
       expect(context.normalize(r'c:\.'), r'c:\');
+      expect(context.normalize(r'c:\foo\.'), r'c:\foo');
       expect(context.normalize(r'B:\.\'), r'B:\');
       expect(context.normalize(r'\\server\share\.'), r'\\server\share');
       expect(context.normalize(r'.\.'), '.');
@@ -351,6 +352,7 @@
       expect(
           context.normalize(r'\\server\share\..\../..\a'), r'\\server\share\a');
       expect(context.normalize(r'c:\..'), r'c:\');
+      expect(context.normalize(r'c:\foo\..'), r'c:\');
       expect(context.normalize(r'A:/..\..\..'), r'A:\');
       expect(context.normalize(r'b:\..\..\..\a'), r'b:\a');
       expect(context.normalize(r'b:\r\..\..\..\a\c\.\..'), r'b:\a');