On 24/12/09 11:51, Jim Meyering wrote:
Pádraig Brady wrote:
On 23/12/09 21:19, Jim Meyering wrote:
Here's a bigger patch (superset of the above) that also makes it so
fremote can be used without those annoyingly unreadable in-function #ifdefs.
It moves the declaration of fremote "up" to precede the first use
and ensures that it's defined to "false" in the !HAVE_INOTIFY case.
That makes it so the two uses work properly when !HAVE_INOTIFY.

Looks good.

Feel free to push.

I added the log and went to do that,
but realized you hadn't pushed yours yet.

I've pushed now.
I was going to push yours but noticed that
you'll need to move File_spec.remote outside
of #if HAVE_INOTIFY.

cheers,
Pádraig.


Reply via email to