[PEPr] Comment on File System::File_Mogile
| From: | Steve Williams | Date: | Fri, 01 Feb 2008 01:04:00 +0000 |
| Subject: | [PEPr] Comment on File System::File_Mogile | ||
| References: | 1 | Groups: | php.pear.dev |
| Request: | Send a blank email to pear-dev+get-49053@lists.php.net to get a copy of this message | ||
Steve Williams (http://pear.php.net/user/sbw) has commented on the proposal for File
System::File_Mogile.
Comment:
Thanks for your feedback. I've updated the package per your comments
except as noted below.
- I'll add the stream wrapper to the wishlist.
- Chuck Burgess"Check your license tags... they link to BSD, but docblock
text says PHP license." I didn't see "PHP" in the license tag, but I was
missing the license name. I added "BSD" as the license name.
- Removed @access, as phpdoc picks it up from the code. I'll put it back
in if you all prefer it.
- Philippe Jausions: "Why use microseconds as timeouts if they're going
to be divided by 1000000?" There are two timeouts, one passed to a
function that takes float seconds and the other to a function that takes
separate seconds and microseconds args. I want to be more consistent than
that, so I chose microseconds. If I change it to seconds, then somebody
would be right to ask, "Why seconds, if you're just going to multiply by
100000?"
Proposal information:
http://pear.php.net/pepr/pepr-proposal-show.php?id=528
--
Sent by PEPr, the automatic proposal system at http://pear.php.net