I started by downloading your .pl code and then generating the HTML. I only looked at the HTML doc, not your code in this review. Looking at the code would perhaps solve lots of questions, but that is not the point of a doc review.
I don't understand the main purpose of this module other than it has something to do with randomizing things. Randomize what? And why? How does your module do much more than obvious uses of the standard Perl rand functions?
Why is there a separate tiny_rand() from instant_rand()? Why the need for 2 functions? As for naming why wouldn't a "tiny" one be "instant"?
As a naive person to your topic, I just do not understand what this thing is supposed to do.
I am not trying to "rain on your parade". But, I think the doc's could be improved. superfluous comment deleted.
update: Starting to look at the actual code and I agree with Re: RFC: Proofread POD for my main random module, please? (more) from pyrt in regards to "tiny_rand()" and "instant_rand()". I would think random_from_list() as pyrt suggests is more understandable. I don't see a reason to expose the 2 parameter version of "instant_rand()" as "tiny_rand()" unless there is some significant performance reason to do so.
In reply to Re: RFC: Proofread POD for my main random module, please? (more)
by Marshall
in thread RFC: Proofread POD for my main random module, please? (more)
by Lady_Aleena
| For: | Use: | ||
| & | & | ||
| < | < | ||
| > | > | ||
| [ | [ | ||
| ] | ] |