This is legacy code and I remember at the time there was a reason I did both. I think you're right though. I have to see how I'm doing this in the new way.

Update: I remember why. CGI->param does not return multi-valued options, unless I'm missing the correct way to do it. I'm not sure why using one would impact performance in any way. Aren't they both already instantiated in memory? My guess would be that I should use the CGI->Vars and its keys.