Re: [PATCH updated] Support for filesystem watching (inotify)

[Paul Eggert]

On 07/06/11 06:36, Stefan Monnier wrote:

> My preference would be to wait for 24.2 since we're in feature freeze,
> but it's the kind of feature which benefits from being available early
> so other packages can start using it.  WDYT?

I briefly looked at it for integer-overflow issues, and found one: it
assumes that inotify cookies fit into an Emacs fixnum.  This assumption
isn't true on 32-bit hosts, unless Emacs is configured with
--with-wide-int.  As you know I'm a fan of wide integers, and my preferred
solution would be to make --with-wide-int the default, which would solve
the problem.  As that is also being considered for 24.2, perhaps the
inotify feature should wait for 24.2 as well.