Picking nits, I know, but how about adding undef, undef as required? I hate to see values assigned to variables that won't be used.