|Just another Perl shrine|
If I had been there, the conversation would have gone something like this.
name used only once
You are either putting data into a variable and not getting it out again (which seems pointless) or trying to get data out of a variable that you haven't put data into (which seems stupid).
global symbol requires package reference, things like that
Your code has variables that can be (accidently!) used outside of the necessary scope. This is dangerous.
It cluttered up the SYSOUT and made it hard to find the real bugs
This are the real bugs. Fix them.
"The first rule of Perl club is you do not talk about