Re: [PEPr] +1 for Web Services::Services_YouTube

On 25 Aug 2006 13:00:22 -0000, Pierre-Alain Joye <pierre.php@gmail.com> wrote:

Pierre-Alain Joye (http://pear.php.net/user/pajoye) has voted +1 on the proposal for Web Services::Services_YouTube. Proposal information: http://pear.php.net/pepr/pepr-proposal-show.php?id=435 Vote information: http://pear.php.net/pepr/pepr-vote-show.php?id=435&handle=pajoye Comment: Some small CS/docs issues, however nothing that cannot be fixed in cvs: About Matthew's comment, I'm not sure we still have to prefix private/protected with underscores. One thing to solve prior to commit in the location of the files. I do not think you need exceptions for this package, also why having an empty extended exception? If a method can throw an exceptions, it should be documented (throw tag).

An empty extended exception allows programs to catch exceptions for this package specifically. It is a requirement when using exceptions to use an exception class specific to the package, whether or not it includes extra functionality. -- Justin Patrin