Vote Details for "HTTP_FloodControl" by cweiske

» Details
Voter: Christian Weiske [Wishlist] Vote: +1 (conditional) Reviews: Cursory source review
» Comment
Nice package. Next time, please provide .phps files for your sources. - Make your private variables public unless you want to explicitely forbid using them in derived classes. - Always add a space between catch and the opening brace. - The constructor docblock has a typo in HTTP_FloodControl_Container. - set() in the file container has a typo in the exception text - When throwing an exception based on a PEAR_Error, also pass the error code as second parameter to the exception constructor.