On Fri, Oct 10, 2008 at 5:01 PM, Shawn O. Pearce <spearce@xxxxxxxxxxx> wrote: > Giuseppe Bilotta <giuseppe.bilotta@xxxxxxxxx> wrote: >> +sub validate_action { >> + my $input = shift || return undef; >> + return undef unless exists $actions{$action}; >> + return $action; >> +} > > Shouldn't $action here be $input? Urgh. Yes, yes it should. Good catch. Should I resend? -- Giuseppe "Oblomov" Bilotta -- 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