[PEPr] +1 for Web Services::Services_YouTube
| From: | Pierre-Alain Joye | Date: | Fri, 25 Aug 2006 13:00:22 +0000 |
| Subject: | [PEPr] +1 for Web Services::Services_YouTube | ||
| References: | 1 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-43800@lists.php.net to get a copy of this message | ||
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).
Another typo: RespnseFormat
--
Sent by PEPr, the automatic proposal system at http://pear.php.net