[PEPr] Comment on Audio::CAPTCHA

Tom Harwood (http://pear.php.net/user/tommeh) has commented on the proposal for Audio::CAPTCHA. Comment: I have added the constructor, updated the license version, changed print to echo, and stopped extending PEAR, I assumed it was required in some way. The WAV file size generated for 5 characters is around 60kb. MP3 output is under research, but it may be be prohibitive because of the compression algorithm complexity involved; this is why initially WAV was chosen since it is universal and simple to maniuplate. With regards to Text_CAPTCHA, I understand your comments as two ideas: 1) Change this proposal to Text_CAPTCHA_Driver_Audio (Text_CAPTCHA_Driver_* is required by the Text_CAPTCHA factory). 2) Keep this proposal as Audio_CAPTCHA and change the method names so they mirror Text_CAPTCHA as closely as possible; then have a Text_CAPTCHA_Driver_Audio which delegates methods required by Text_CAPTCHA to Audio_CAPTCHA. I prefer #2 as gives the flexibility of requiring Text_CAPTCHA (and its dependencies, including GD lib) if you want to use it that way, or not requiring Text_CAPTCHA and using it standalone. Do you have any preference between #1, #2, or did you mean something else? Thanks for the comments :) Proposal information: http://pear.php.net/pepr/pepr-proposal-show.php?id=433 -- Sent by PEPr, the automatic proposal system at http://pear.php.net