Junio C Hamano wrote: > Jakub Narebski <jnareb@xxxxxxxxx> writes: > >> While at it make git_get_references return hash in list context, >> and reference to hash (as it used to do) in scalar and void contexts. > > Why did you have to add this? Generally, context sensitive > returns make the program much harder to maintain, because it > forces the call sites to be extremely careful to choose between > "my ($foo) = func()" vs "my $foo = func()", and people who later > call the function inevitably make mistakes. > > So unless there is a compelling reason, I'd rather not see more > "wantarray" in the program. O.K. I have browsed through gitweb, and I see that we almost always want the hashref, not hash (for passing further). -- Jakub Narebski Poland - To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html