inotify: repowatcher: don't use a watches attribute to count watches The encapsulated watcher alread has a __len__ method for this purpose. Also, repowatcher.watches was never decremented on watch deletions.

inotify: put the "while True: poll()" loop in pollable class

inotify: refactor (un)register methods into pollable object repowatcher.master becomes unnecessary

inotify.server: the decorator eventaction() shouldn't be a method of repowatcher

inotify: add log config option redirect inotify server output to a file

cmdutil: service: logfile option to redirect stdout & stderr in a file

inotify: raise QueryFailed when the server crash When the server crashes, it does not write back on the socket. The message which is then read from the socket is '', which raises a TypeError. Catching TypeError to raise QueryFailed instead.

inotify: server: move split() out of server split() has nothing to do with the server logic, it does not need to be in the class. Move it on top, next to join() which does the opposite.

inotify: remove unused variables

atomictempfile: fix exception in __del__ if mktempcopy fails (self._fp is None) Make what is going on more obvious by explicitely getting the 'closed' attribute from _fp, instead of from the gettatr proxy.