<nitpicking>
Also, the last expression in you module should be:

1;
[download]

(or something similar that evaluates to 'true'), otherwise you'll get a module-'did not return a true value'-error when you 'use' it.
</nitpicking>