[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Added basic file system watching support.
From: |
Eli Zaretskii |
Subject: |
Re: [PATCH] Added basic file system watching support. |
Date: |
Tue, 11 Dec 2012 10:29:14 +0200 |
> From: Michael Albinus <address@hidden>
> Cc: address@hidden, address@hidden, address@hidden, address@hidden
> Date: Tue, 11 Dec 2012 09:19:10 +0100
>
> >> Could we extend the interface of `inotify-rm-watch' to add a file name?
> >
> > Why do you need an extension? This is Lisp: we could pass the file
> > name _as_ the descriptor. Since the remote file handler will take
> > care of the call anyway, it will assign the correct meaning to the
> > descriptor.
>
> Sure, that would be sufficient for Tramp. I had the impression, that a
> more complex structure as descriptor was chosen, because there could be
> several `inotify-add-watch' calls for the *same* file, being different
> in the aspect or the callback function.
Then Tramp could maintain its own data structure for watched files,
and give each watch a unique identifier, like (FILENAME . NUMBER), or
just NUMBER, or whatever.
IOW, the DESCRIPTOR is an opaque data type, which the implementation
back-end, and the back-end alone, can and should interpret.
- Re: [PATCH] Added basic file system watching support., Michael Albinus, 2012/12/11
- Re: [PATCH] Added basic file system watching support., Eli Zaretskii, 2012/12/11
- Re: [PATCH] Added basic file system watching support., Michael Albinus, 2012/12/11
- Re: [PATCH] Added basic file system watching support., Michael Albinus, 2012/12/11
- Re: [PATCH] Added basic file system watching support., Eli Zaretskii, 2012/12/11
- Re: [PATCH] Added basic file system watching support., Michael Albinus, 2012/12/11
- Re: [PATCH] Added basic file system watching support., Eli Zaretskii, 2012/12/11
- Re: [PATCH] Added basic file system watching support., Leo, 2012/12/11
- Re: [PATCH] Added basic file system watching support., Eli Zaretskii, 2012/12/11