{"action":"created","issue":{"url":"https://api.github.com/repos/async-interop/event-loop/issues/58","repository_url":"https://api.github.com/repos/async-interop/event-loop","labels_url":"https://api.github.com/repos/async-interop/event-loop/issues/58/labels{/name}","comments_url":"https://api.github.com/repos/async-interop/event-loop/issues/58/comments","events_url":"https://api.github.com/repos/async-interop/event-loop/issues/58/events","html_url":"https://github.com/async-interop/event-loop/issues/58","id":156375494,"number":58,"title":"Operations on Invalid Watchers Must Throw","user":{"login":"bwoebi","id":3154871,"avatar_url":"https://avatars.githubusercontent.com/u/3154871?v=3","gravatar_id":"","url":"https://api.github.com/users/bwoebi","html_url":"https://github.com/bwoebi","followers_url":"https://api.github.com/users/bwoebi/followers","following_url":"https://api.github.com/users/bwoebi/following{/other_user}","gists_url":"https://api.github.com/users/bwoebi/gists{/gist_id}","starred_url":"https://api.github.com/users/bwoebi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bwoebi/subscriptions","organizations_url":"https://api.github.com/users/bwoebi/orgs","repos_url":"https://api.github.com/users/bwoebi/repos","events_url":"https://api.github.com/users/bwoebi/events{/privacy}","received_events_url":"https://api.github.com/users/bwoebi/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/async-interop/event-loop/labels/discussion","name":"discussion","color":"cc317c"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":12,"created_at":"2016-05-23T21:54:23Z","updated_at":"2016-05-27T13:05:55Z","closed_at":null,"body":"From #53:\r\n\r\n> May we please add that cancel()'ling multiple times or any other operation on already invalid watchers is disallowed (as watcher ids may be reused)?\r\nCalling unreference(), reference(), enable() and disable() multiple times must not error.\r\n\r\nIs everyone okay with that (I'll open a PR tomorrow to add that in docblocks)? Or are there issues here?"},"comment":{"url":"https://api.github.com/repos/async-interop/event-loop/issues/comments/222141145","html_url":"https://github.com/async-interop/event-loop/issues/58#issuecomment-222141145","issue_url":"https://api.github.com/repos/async-interop/event-loop/issues/58","id":222141145,"user":{"login":"joshdifabio","id":5609851,"avatar_url":"https://avatars.githubusercontent.com/u/5609851?v=3","gravatar_id":"","url":"https://api.github.com/users/joshdifabio","html_url":"https://github.com/joshdifabio","followers_url":"https://api.github.com/users/joshdifabio/followers","following_url":"https://api.github.com/users/joshdifabio/following{/other_user}","gists_url":"https://api.github.com/users/joshdifabio/gists{/gist_id}","starred_url":"https://api.github.com/users/joshdifabio/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/joshdifabio/subscriptions","organizations_url":"https://api.github.com/users/joshdifabio/orgs","repos_url":"https://api.github.com/users/joshdifabio/repos","events_url":"https://api.github.com/users/joshdifabio/events{/privacy}","received_events_url":"https://api.github.com/users/joshdifabio/received_events","type":"User","site_admin":false},"created_at":"2016-05-27T13:05:55Z","updated_at":"2016-05-27T13:05:55Z","body":"That's true, although the driver implementations will be in packages which depend on this package anyway, so I don't know how much difference that would really make. Applying GRASP would seem to suggest that it would belong in the `Loop` class at least.\r\n\r\nJust so I understand, are you thinking that watcher ID's returned by `Loop` _should_ be unique for the duration of the process, or just unique to the current driver?"}}
{"actor":{"display_login":"joshdifabio"}}