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

Issue 10867082: Fix apidoc generation of pkg on Windows (Closed)

Created:
8 years, 3 months ago by Johnni Winther
Modified:
8 years, 3 months ago
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Fix apidoc generation of pkg on Windows Committed: https://code.google.com/p/dart/source/detail?r=11431

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+2 lines, -2 lines) Patch
M utils/apidoc/apidoc.dart View 1 chunk +2 lines, -2 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Johnni Winther
This fixes a path bug occurring on Windows.
8 years, 3 months ago (2012-08-27 07:55:54 UTC) #1
Jennifer Messerly
lgtm, but I'm surprised that DirectoryLister returns paths that aren't in native form already? It ...
8 years, 3 months ago (2012-08-27 19:40:41 UTC) #2
Jennifer Messerly
8 years, 3 months ago (2012-08-27 23:00:25 UTC) #3
On 2012/08/27 19:40:41, John Messerly wrote:
> lgtm, but I'm surprised that DirectoryLister returns paths that aren't in
native
> form already? It seems like once I give a Path to dart:io, any paths it
creates
> should stay in "normalized form" ... if not, maybe we should rename "new
> Path.fromNative" to "new Path" and do something else with the current
> constructor. This seems like it will be a common mistake.

or DirectoryLister could give you Path objects... that's probably slower though.

Powered by Google App Engine
This is Rietveld 408576698