From: Krzesimir Nowak Date: Thu, 05 Dec 2013 09:16:14 GMT Subject: Re: [PATCH 1/5] gitweb: Add a comment explaining the meaning of $/ Message-ID: <1386234974.2186.1.camel@localhost.localdomain> In-Reply-To: On Wed, 2013-12-04 at 12:28 -0800, Junio C Hamano wrote: > Martin Langhoff writes: > > > On Wed, Dec 4, 2013 at 10:46 AM, Krzesimir Nowak wrote: > >> On Wed, 2013-12-04 at 16:11 +0100, Jakub Narębski wrote: > >>> On Wed, Dec 4, 2013 at 2:42 PM, Krzesimir Nowak wrote: > >>> > >>> > So future reader will know what does it mean without running "perldoc > >>> > perlvar". > >>> > >>> Hmmm... shouldn't future reader know it anyway? It is not that cryptic. > >>> I'd say it is idiomatic Perl. > >> > >> It's plainly obscure. And I think it is not that often used - > > > > It's classic Perl. > > > > Perhaps you'd want to "use English;" and call it > > $INPUT_RECORD_SEPARATOR in a patch titled "Make things readable to > > non-Perl natives". > > Hmm, but do we want to see "use English" there in the first place? Nevermind, I'm going to drop that patch. -- Krzesimir Nowak Software Developer Endocode AG krzesimir@endocode.com ------ Endocode AG, Johannisstraße 20, 10117 Berlin info@endocode.com | www.endocode.com Vorstandsvorsitzender: Mirko Boehm Vorstände: Dr. Karl Beecher, Chris Kühl, Sebastian Sucker Aufsichtsratsvorsitzende: Jennifer Beecher Registergericht: Amtsgericht Charlottenburg - HRB 150748 B