Re: [Nautilus-list] [PATCH]: Avoid directory reload when FAM is active.

[Yoann Vandoorselaere <yoann mandrakesoft com>]

Darin Adler <darin bentspoon com> writes:

> On Sunday, June 17, 2001, at 05:11  AM, Yoann Vandoorselaere wrote:
> 
> > +        use_monitoring = nautilus_monitor_active ();
> 
> Just checking if the monitor is active is not enough. You also have to
> check that the directory in question is a "file://" directory and not
> another kind of URL. Also, you have to explicitly check that this is
> not the reload case, since reload won't actually do any reloading
> without the code, and we do want reload to cause a reload even with
> the file monitoring on. So we need 3 checks, not just this single one.
> 
>      -- Darin
> 
> PS: I know this is going to drive Yoann nuts, but I started working on
> this fix myself after the discussion last week -- I had forgotten that
> Yoann said he would do it. I probably have the fix done correctly, but
> I haven't had a chance to check it in, and won't have a chance to do
> it until I return from my trip.

BTW : Instead of fixing what I told I would fix (and you know I always
      do what I say), try to focus on the question that I sent you and
      that prevent me to fix the *important* issue that are sleeping in
      Nautilus and that you aren't working on.

/me upset

-- 
Yoann Vandoorselaere | - Windows  Where do you want to go today?
MandrakeSoft         | - MacOS    Where do you want to be tomorrow?
                     | - Linux    Are you coming, or what?  -- Bill Maidment