Re: [evolution-patches] fix for #300217 (camel/providers/local)



On Tue, 2005-09-06 at 16:59 +0800, Not Zed wrote:
> Patch doesn't do what you say it will.
I did not understand how? The earlier code was just checking for first
character of the absolute new name, which would work only for the
folders created at top level, I am checking for the new name given by
user.

> 
> strstr?  sheesh, that's inefficient for looking for a char, you should
> use strchr, infact you should use strrchr in this case, you want the
> last one.
Oh, wondering how/why I ended up using that, I will change it, thanks!.

> IMO, '.' names are perfectly valid anyway and there's no reason to hide
> them.  The linux 'convention' is just a hack.
Then looks like I should be working in another direction, I will see
how/whether we can show hidden dirs in folder tree. Please drop some
idea if you already have one.
	
Thanks,
Vivek Jain



[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]