pre-cleanup windows fix

Author: jcollins-g

lib/src/model/package_graph.dart

@@ -22,7 +22,7 @@ import 'package:dartdoc/src/render/renderer_factory.dart';
 import 'package:dartdoc/src/special_elements.dart';
 import 'package:dartdoc/src/tuple.dart';
 import 'package:dartdoc/src/warnings.dart';
-import 'package:glob/glob.dart';
+import 'package:dartdoc/src/model_utils.dart' show matchGlobs;
 
 class PackageGraph {
   PackageGraph.UninitializedPackageGraph(
@@ -964,9 +964,7 @@ class PackageGraph {
       // might not be where the element was defined, which is what's important
       // for nodoc's semantics.
       List<String> globs = config.optionSet['nodoc'].valueAt(file.parent);
-      _configSetsNodocFor[fullName] = globs.any((g) =>
-          Glob(resourceProvider.pathContext.canonicalize(g))
-              .matches(resourceProvider.pathContext.canonicalize(fullName)));
+      _configSetsNodocFor[fullName] = matchGlobs(globs, fullName);
     }
     return _configSetsNodocFor[fullName];
   }

lib/src/model_utils.dart

@@ -11,9 +11,42 @@ import 'package:analyzer/dart/ast/ast.dart';
 import 'package:analyzer/dart/element/element.dart';
 import 'package:analyzer/src/dart/ast/utilities.dart';
 import 'package:dartdoc/src/model/model.dart';
+import 'package:path/path.dart' as path;
+import 'package:glob/glob.dart';
+
+final _driveLetterMatcher = RegExp(r'^\w:\\');
 
 final Map<String, String> _fileContents = <String, String>{};
 
+/// This will handle matching globs, including on Windows.
+///
+/// Assumes that globs and resource provider are from the same drive, which
+/// will be the case for globs relative to dartdoc_options.yaml.
+///
+/// On windows, globs are assumed to use Windows paths in combination with
+/// globs, e.g. `C:\foo\bar\*.txt`.
+bool matchGlobs(List<String> globs, String fullName, {bool isWindows}) {
+  isWindows ??= Platform.isWindows;
+  var filteredGlobs = <String>[];
+
+  if (isWindows) {
+    assert(_driveLetterMatcher.hasMatch(fullName),
+        'can not find drive letter in $fullName');
+    // TODO(jcollins-g): handle globs referencing different drives?
+    fullName = fullName.replaceFirst(_driveLetterMatcher, r'\');
+    for (var glob in globs) {
+      // `C:\` => `\` for rejoining via posix.
+      glob = glob.replaceFirst(_driveLetterMatcher, r'/');
+      filteredGlobs.add(path.posix.joinAll(path.windows.split(glob)));
+    }
+  } else {
+    filteredGlobs.addAll(globs);
+  }
+
+  return filteredGlobs.any((g) =>
+      Glob(g, context: isWindows ? path.windows : null).matches(fullName));
+}
+
 /// Returns the [AstNode] for a given [Element].
 ///
 /// Uses a precomputed map of [element.source.fullName] to [CompilationUnit]

test/model_utils_test.dart

@@ -8,6 +8,18 @@ import 'package:dartdoc/src/model_utils.dart';
 import 'package:test/test.dart';
 
 void main() {
+  group('match glob', () {
+    test('basic POSIX', () {
+      expect(
+          matchGlobs(['/a/b/*', '/b/c/*'], '/b/c/d', isWindows: false), isTrue);
+      expect(matchGlobs(['/q/r/s'], '/foo', isWindows: false), isFalse);
+    });
+
+    test('basic Windows', () {
+      expect(matchGlobs([r'C:\a\b\*'], r'c:\a\b\d', isWindows: true), isTrue);
+    });
+  });
+
   group('model_utils stripIndentFromSource', () {
     test('no indent', () {
       expect(stripIndentFromSource('void foo() {\n  print(1);\n}\n'),