[PEPr] Comment on Text::PathArray

From: Date: Mon, 08 May 2006 22:32:17 +0000
Subject: [PEPr] Comment on Text::PathArray
References: 1  Groups: php.pear.dev 
Request: Send a blank email to pear-dev+get-42475@lists.php.net to get a copy of this message
Denny Shimkoski (http://pear.php.net/user/denny) has commented on the proposal for Text::PathArray. Comment: I've update the class according to the most recent feedback and posted it here: http://bytebrite.com/pear/path.html Thanks for the tips, Bertrand. I've changed the license to MIT and formatted the source using php -s. As for why it should be in Text, I couldn't think of a more logical place to put it. As for the lack of interest, you may be right. However, it wouldn't hurt to have this little utility around when you need it. Furthermore, I don't feel that including it in the repository will hurt PEAR's image. So, why not? Martin, you sold me on the name change. It is now Text_PathNavigator. I also added support for iterators, but indirectly. Basically, the user has to call $it = $p->getIterator() and then operate on $it, which is an ArrayIterator instance. The reason I did it this way is because I didn't want to force PHP5 on users, and, as far as I know, the implements keyword would cause PHP4 to choke. Am I mistaken on this point? Or can you think of a clever workaround, aside from simply defining two distinct classes? Thanks again, Denny Proposal information: http://pear.php.net/pepr/pepr-proposal-show.php?id=393 -- Sent by PEPr, the automatic proposal system at http://pear.php.net

« previous php.pear.dev (#42475) next »