C'mmon ppl, I know you can do better than that.

cdherold: you should keep you declarations up top. So lower the for-loop below the use.

Furthermore, keep the sub routine outside the loops. Clean up that one, so it doesn't depend on vars outside the sub (is a good idea in general)

Look at the MAIL pipe. I don't think open closes MAIL before opening it, but I'm not sure. So *always* check whether a open fails with:

open BLAH, "some_expr" or die "Could not do some_expr: $!";
[download]

The behaviour cdherold describes could be well due pipe issues. The first print has some effect, but the others are just added to the body or printed to void.

Well, these are quickies as well. But should give cdherold a start.

Cheers,

Jeroen
"We are not alone"(FZ)