On Sat, Nov 7, 2009 at 10:44 AM, Pierrick Charron <pierr...@webstart.fr>wrote:
> Hi, > > I created the documentation related to the stomp extension (I released > yesterday the version 0.3.0 of it) and i would like to get some > feedback on this documentation before pushing it online. > > You'll find the documentation here : http://php.adoy.net/stomp > And the XML here : http://www.adoy.net/stomp/doc.tgz > > Thanks > Pierrick > Hi Pierrick Requirements ============ If you want more people to participate in testing/using the extension, I think you should try to do more regression tests against lower php versions like 5.2.4 Most people using PHP are still behind in terms of what they are using the thier PROD environments. Installation ============= You should make the installation instructions clearer instead of sending them to another page Something like: ./configure --enable-stomp OR pecl install stomp-beta should be good enough. I also thing you should prevent implicit conversion of (void *) when performing dynamic memory allocation. It may cause the C++ based compilers to fail on Windows. Examples ============ I prefer to use more realistic examples instead of "foo" and "bar". PHP is used all over the world. Some people may not really follow. -- "Good Enough" is not good enough. To give anything less than your best is to sacrifice the gift. Quality First. Measure Twice. Cut Once.