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

Issue 8244001: Clean up directory listing interface. (Closed)

Created:
9 years, 2 months ago by Mads Ager (google)
Modified:
9 years, 2 months ago
Reviewers:
Søren Gjesse
CC:
reviews_dartlang.org, vm-dev_dartlang.org
Visibility:
Public.

Description

Clean up directory listing interface. Remove open and close. A directory instance just holds a string representing the path to the directory. R=sgjesse@google.com BUG= TEST= Committed: https://code.google.com/p/dart/source/detail?r=362

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+185 lines, -285 lines) Patch
M runtime/bin/builtin_in.cc View 1 chunk +1 line, -3 lines 0 comments Download
M runtime/bin/directory.h View 1 chunk +4 lines, -7 lines 0 comments Download
M runtime/bin/directory.cc View 2 chunks +10 lines, -42 lines 0 comments Download
M runtime/bin/directory.dart View 2 chunks +13 lines, -21 lines 2 comments Download
M runtime/bin/directory_impl.dart View 1 chunk +71 lines, -101 lines 0 comments Download
M runtime/bin/directory_linux.cc View 7 chunks +29 lines, -37 lines 0 comments Download
M runtime/bin/directory_macos.cc View 7 chunks +32 lines, -37 lines 0 comments Download
M runtime/bin/directory_win.cc View 6 chunks +22 lines, -33 lines 0 comments Download
M runtime/tests/dart/src/DirectoryTest.dart View 2 chunks +3 lines, -4 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
Mads Ager (google)
9 years, 2 months ago (2011-10-12 08:00:27 UTC) #1
Søren Gjesse
lgtm http://codereview.chromium.org/8244001/diff/1/runtime/bin/directory.dart File runtime/bin/directory.dart (right): http://codereview.chromium.org/8244001/diff/1/runtime/bin/directory.dart#newcode40 runtime/bin/directory.dart:40: * the listing operation completed. What does "not ...
9 years, 2 months ago (2011-10-12 09:30:30 UTC) #2
Mads Ager (google)
9 years, 2 months ago (2011-10-12 10:36:28 UTC) #3
http://codereview.chromium.org/8244001/diff/1/runtime/bin/directory.dart
File runtime/bin/directory.dart (right):

http://codereview.chromium.org/8244001/diff/1/runtime/bin/directory.dart#newc...
runtime/bin/directory.dart:40: * the listing operation completed.
On 2011/10/12 09:30:30, Søren Gjesse wrote:
> What does "not completed" actually mean? Will the first error trigger done?
Will
> recurse continue even if some sub-directories cannot be opened?

Good question. :)

Currently, done means that no errors that could effect the list of files and
directories occurred. Therefore, the listing is done and it completed (so you
got all the callbacks for everything that was encountered). If, for instance, we
found a directory that could not be opened, we don't list the contents of that
directory and completed will be false.

Powered by Google App Engine
This is Rietveld 408576698