[PEPr] +1 for HTML::HTML_Entities

Till Klampaeckel (http://pear.php.net/user/till) has voted +1 on the proposal for HTML::HTML_Entities. Proposal information: http://pear.php.net/pepr/pepr-proposal-show.php?id=546 Vote information: http://pear.php.net/pepr/pepr-vote-show.php?id=546&handle=till This vote is conditional. The condition is: First off, very nice code, clean and well documented and also tests! ;-) But, here are my reasons that make this vote conditional: 1) Don't use the PHP License (as it conflicts with GPL), please select LGPL or The New BSD License instead. I personally favour the latter. 2) Please create your own Exception class (extending PEAR_Exception - HTML_Entities_Exception). 3) You probably don't need another tests/examples dir here: test /usr/local/share/pear/tests/HTML_Entities/tests/html_entities1.phpt test /usr/local/share/pear/tests/HTML_Entities/tests/html_entities2.phpt test /usr/local/share/pear/tests/HTML_Entities/tests/html_entities3.phpt doc /usr/local/share/doc/pear/HTML_Entities/examples/html_entities_example.php php /usr/local/share/pear/HTML/Entities.php The second "tests" and "examples" is redundant. I think you can define this in your package.xml. 4) I think you need to define a replacement for @package_version@ in your package.xml. After I installed your package it still said @package_version@ in there. -- Sent by PEPr, the automatic proposal system at http://pear.php.net