Eeeek. That code would've been terrible practice in 2000 when I first started writing Perl :(<p>The Mysql module was deprecated in favour of the unified DBI interface, cgi-lib.pl was a <i>perl4</i> library that was replaced by a module called CGI (shipped with perl5 itself from 5.4+).<p>Calling functions with & is a perl4-ism and almost always the wrong thing to do in perl5, and C-style for is silly in most cases in Perl - for example; the loops in there would likely be better redone as foreach loops, or possibly just a grep.<p>And, of course, YAY not using placeholders in the SQL.<p><i>sigh</i>