That makes more sense. Memoize will only cache the arguements to the function, and the return value so you are fine with your implementation. Sorry for jumping on this, but I thought that you might be doing something like the following:

use Digest::MD5 qw(md5);
use Memoize;
memoize('md5');
[download]

This would use gobs of memory (depending on the input) and wouldn't really accomplish anything useful.

Do I get an award for coming up with the most unproductive use of Memoize???