1) Leaving the instantiation of LWP::UserAgent outside of the while loop is better because it is a reusable object. Multiple instantiations are unecessary and wasteful.

2) just add $| = 1; to the top of your code and be done with it. Normally, buffering is not an issue with most scripts, but sophisticated network I/O should turn buffering off (same as auto-flushing on) because you don't won't to be stuck waiting for your data to be sent to the server.

jeffa

L-LL-L--L-LL-L--L-LL-L--
-R--R-RR-R--R-RR-R--R-RR
F--F--F--F--F--F--F--F--
(the triplet paradiddle)