Junio C Hamano <gitster@xxxxxxxxx> writes:
> Jakub Narebski <jnareb@xxxxxxxxx> writes:
>
>> The idea is NOT to use subroutine prototypes to create new syntax;
>> prototypes are to be purely informational and optional.
That's unfortunately a grave misconception, isn't it?
For example, can you explain (1) how these three calls behave before
running them, and (2) why these three behave the way they do?
sub foo { my ($s, %f) = @_; print "s = $s\n"; }
sub bar ($;%) { my ($s, %f) = @_; print "s = $s\n"; }
my @it = ('This is my string');
my %hash = (rose => 'blue', violet => 'green');
foo @it, %hash; # call 1
bar @it, %hash; # call 2
bar $it[0], %hash; # call 3
By adding ($;%) to an existing function that did not have prototype, you
changed the semantics of the function and:
(1) it is your responsibility to make sure you did not break existing
callers when you made such a change, and
(2) programmers who want to call any existing function in your program
need to check how the function groks its parameters and make sure
they do not fall into the same pitfalls as the call sites you had to
fix in step (1). They cannot rely on the old fashioned "arguments
are passed as a flattened list" idiom anymore before checking if you
have prototypes to the function they want to call.
Prototypes used carelessly tend to force users to do unnecessary things,
because the caller cannot rely on the old fashioned "arguments are passed
as a flattened list" semantics and check how each and every function is
prototyped before making a call.
I am not saying that Perl prototypes is a bad thing. The point of the
prototype is to change the syntax and semantics so that you can write a
function to which arguments are _not_ passed as a flattend list, and
without them you cannot write something that emulates "push @a, $b, $c".
But you need to be aware of what it does to your callers.
--
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
