One option may be to move all the %mail declarations into an array-of-hash-references or a hash-of-hash references. Then, your if-elsif-else can be converted into a CASE or SWITCH type block which sets a reference to the hash you want to send to sendmail(). This will make your script a little easier to read and make it easier to add items to the CASE/SWITCH by adding more logic and additional entries into the hash.
OTOH, as someone once said: If it ain't baroque, don't fix it. Not to be facetious, but is this script having performance or resource problems? Security issues? Limited features? I understand the need for efficiency and I respect slick coding as much as the next monk, but unless you have a good reason, why modify something that you have been using for a year(in a production environment I assume) unless you want to expand it's functionality or correct a flaw? Just a thought.
Later
In reply to Re: Advise on current Mail script
by pzbagel
in thread Advise on current Mail script
by Anonymous Monk
| For: | Use: | ||
| & | & | ||
| < | < | ||
| > | > | ||
| [ | [ | ||
| ] | ] |