Skip to content

Commit

Permalink
[io grant] Remove IO::Path.watch
Browse files Browse the repository at this point in the history
There are zero tests for it and the whole notifications business
is quite rough on the edges and probably needs more work. Remove
the method so we don't lock ourselves into supporting some API
that might not be suitable once notifications are sorted out.

The functionality is still available via
IO::Notification.watch-path: $path
  • Loading branch information
zoffixznet committed May 28, 2017
1 parent f0eb245 commit 21fd2c4
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions src/core/IO/Path.pm
Expand Up @@ -213,12 +213,6 @@ my class IO::Path is Cool does IO {
multi method IO { self }
method open(IO::Path:D: |c) { IO::Handle.new(:path(self)).open(|c) }

#?if moar
method watch(IO::Path:D:) {
IO::Notification.watch-path($.absolute);
}
#?endif

proto method absolute(|) { * }
multi method absolute (IO::Path:D:) {
$!abspath //= $!SPEC.rel2abs($!path,$!CWD)
Expand Down

0 comments on commit 21fd2c4

Please sign in to comment.