Re: [PATCH 4/4] gitweb: show notes in commit(diff) view

On Sat, Feb 6, 2010 at 3:34 PM, Jakub Narebski [off-list ref] wrote:

Giuseppe Bilotta wrote:

quoted

2010/2/6 Jakub Narebski [off-list ref]:

quoted

On Thu, 4 Jan 2010, Giuseppe Bilotta wrote:

[...]

quoted

quoted

quoted

 # return all refs matching refs/notes/<globspecs> where the globspecs
 # are taken from the notes feature content.
 sub get_note_refs {
+     local $/ = "";
+

Why it is needed here?  Why you want to use empty lines as terminator
(which means reading whole paragraphs), while treating two or more
consecutive empty lines as a single empty line (according to
perlvar(1))?

If you want to slurp whole file, this should be

       local $/;

or more explicit

       local $/ = undef;

Ah, sorry, for some reason I thought "" was the default.

If you wanted to use default value, why set it at all?

Ach, sorry, forgot to reply to the first part of the question. It's
used in a context where $/ is locally set to \0, so it needs to be
reset.

-- 
Giuseppe "Oblomov" Bilotta