You can't use $1, $2, ... in my(). It would be far better to avoid using $1, $2, ... altogether and use properly named variables that do not have any scoping issues.