I'd like to encourage more descriptive variable names, especially when you end up using the same variable name for two different variable types (sub vs hash). This line is kind of amusing ;)