From: Junio C Hamano <junkio@cox.net>
Date: Thu, 14 Sep 2006 20:13:01 GMT
Subject: Re: [PATCH (take 3)] gitweb: Use File::Find::find in git_get_projects_list
Message-ID: <7vejue2omq.fsf@assigned-by-dhcp.cox.net>
In-Reply-To: <200609142134.33725.jnareb@gmail.com>

Jakub Narebski <jnareb@gmail.com> writes:

> +			wanted => sub {
> +				# skip current directory
> +				return if (m!^/|.|..$!);

Huh?

 (1) Did you mean to say "\." (not any single character but
     literally dot)?

 (2) how does the alternatives within m{} construct bind (iow,
     please be gentle to the readers)?  Do you mean

	return if (/^\/(?:\.|\.\.)$/)

     in other words,

     	return if (/^\/\.$/ || /^\/\.\.$/)

     in other words,

        return if (/^\/\.{1,2}/)

     ???