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

Issue 11661003: Implement toString on File and Directory objects. (Closed)

Created:
8 years ago by Mads Ager (google)
Modified:
8 years ago
Reviewers:
Søren Gjesse
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Implement toString on File and Directory objects. R=sgjesse@google.com BUG=dartbug.com/7552 Committed: https://code.google.com/p/dart/source/detail?r=16436

Patch Set 1 #

Total comments: 3
Unified diffs Side-by-side diffs Delta from patch set Stats (+17 lines, -1 line) Patch
M sdk/lib/io/directory.dart View 1 chunk +5 lines, -0 lines 0 comments Download
M sdk/lib/io/directory_impl.dart View 1 chunk +3 lines, -1 line 2 comments Download
M sdk/lib/io/file.dart View 1 chunk +5 lines, -0 lines 0 comments Download
M sdk/lib/io/file_impl.dart View 1 chunk +2 lines, -0 lines 1 comment Download
M tests/standalone/io/directory_test.dart View 1 chunk +1 line, -0 lines 0 comments Download
M tests/standalone/io/file_test.dart View 1 chunk +1 line, -0 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
Mads Ager (google)
8 years ago (2012-12-21 07:31:07 UTC) #1
Søren Gjesse
lgtm https://codereview.chromium.org/11661003/diff/1/sdk/lib/io/directory_impl.dart File sdk/lib/io/directory_impl.dart (right): https://codereview.chromium.org/11661003/diff/1/sdk/lib/io/directory_impl.dart#newcode231 sdk/lib/io/directory_impl.dart:231: String toString() => "Directory: '$path'"; Are you sure ...
8 years ago (2012-12-21 10:16:00 UTC) #2
Søren Gjesse
lgtm Additional comment maybe drop path and name getters and only have toString.
8 years ago (2012-12-21 10:16:53 UTC) #3
Mads Ager (google)
https://codereview.chromium.org/11661003/diff/1/sdk/lib/io/directory_impl.dart File sdk/lib/io/directory_impl.dart (right): https://codereview.chromium.org/11661003/diff/1/sdk/lib/io/directory_impl.dart#newcode231 sdk/lib/io/directory_impl.dart:231: String toString() => "Directory: '$path'"; On 2012/12/21 10:16:00, Søren ...
8 years ago (2012-12-21 10:35:03 UTC) #4
Mads Ager (google)
8 years ago (2012-12-21 11:00:22 UTC) #5
On 2012/12/21 10:35:03, Mads Ager wrote:
> https://codereview.chromium.org/11661003/diff/1/sdk/lib/io/directory_impl.dart
> File sdk/lib/io/directory_impl.dart (right):
> 
>
https://codereview.chromium.org/11661003/diff/1/sdk/lib/io/directory_impl.dar...
> sdk/lib/io/directory_impl.dart:231: String toString() => "Directory: '$path'";
> On 2012/12/21 10:16:00, Søren Gjesse wrote:
> > Are you sure you want the "Directory: " prefix?
> 
> Good question. I was thinking that it might be helpful for debugging purposes
to
> be able to print some entity and it will tell you what it is and the path.

I'll go ahead and land this for now. We should revisit this when we have a fresh
look at the File and Directory APIs after new years.

Powered by Google App Engine
This is Rietveld 408576698