It's seems a good idea (to me) to use lexical scoping in subroutines by using 'my'. This way, there's less (I'm always afraid to say 'no') chance of a global variable being inadvertantly clobbered.

Another good practice is to use 'strict'. This will save a lot of debugging time and frustration.

Mick