note Anonymous Monk <code> if (open ...) { print error } else { real code here } #EOF </code> <p>is better written as</p> <code> if (open ...) { print error exit(1); } real code here </code> <p>Besides that, you might want to try splitting the script's processing stages into well-named subroutines. It will help you reduce the amount of variables in the current scope, too. And perhaps rid of the temporary file -- I'm sure MIME::Lite can attach a "file" from a string variable.</p> 1013260 1013260 5