[Nautilus-list] Re: [PATCH]: avoid directory reloading when not needed.

[Darin Adler <darin bentspoon com>]

On Thursday, June 14, 2001, at 02:13  PM, Yoann Vandoorselaere wrote:

> What the patch does currently is that it make reloading
> happen either :
>
> - when the refresh button is used.
> - when a new window is opened.
>
> For the second case, it would be good to be able to know
> if a window already display the directory.
>
> Is the patch clean / right enough to commit ?

Hmm. Even though I said "you could try making that change", now that I see 
the actual patch it doesn't seem right to me.

If someone types a new location into an existing window, I don't think the 
behavior should be any different from opening a new window. In either case,
 if the new location happens to be cached, we need to do a new load and 
not rely on old cached data unless we have synchronization like FAM.

A particularly bad case of this has to do with going up from your home 
directory, I seem to recall. There were cases where new files just didn't 
show up without a refresh. We can dig around the bug database to find this 
specific awful case. I'm pretty sure this change reintroduces the bug.

Maybe some of the other folks in the Nautilus community can weigh in on 
this. If there's a consensus among enough of you guys that we should do 
this change, I'll approve it. But I personally think it's not an 
improvement. It brings back the old bug we had where you see cached data 
that confused.

I'm sorry that I encouraged you to write this patch. As I said before: "I'
d prefer to just make the FAM change though, and not waste time optimizing 
things for the non-FAM case." The FAM change will not reintroduce the bug,
 and it's incredibly simple to do.

    -- Darin