Hey guys,

Thank you for the feedback. I will get these done asap and let you know. I
think I have spent more time writing and fixing documentation than writing
the extension itself =)

Pieter


On 6/7/10 10:08 AM, "Philip Olson" <phi...@roshambo.org> wrote:

> 
> 
> On Jun 7, 2010, at 12:49 AM, Hannes Magnusson wrote:
> 
>> > On Mon, Jun 7, 2010 at 08:59, Pieter de Zwart
>> > <pdezw...@rubiconproject.com> wrote:
>>> >> Hey guys,
>>> >>
>>> >> Since build happens Friday, I waited till now so that I can get a solid
>>> week
>>> >> of feedback before this gets included in the main docs. I committed revs
>>> >> 300253 and 300254.
>>> >> I would really appreciate it if someone could give it a look over to make
>>> >> sure I am following standards and procedures.
>>> >> Once that is done, I will finally make release 0.1 of the client... Only
>>> >> about 2 months later than expected.
>> >
>> >
>> > The install section contains a detailed info on how to install the
>> > library dependency, but then says:
>> > +    </programlisting>
>> > +  <para>
>> > +  download and compile the <filename>amqp.so</filename> file and add
>> > +  an <filename>extension</filename> entry in your
>> > <filename>php.ini</filename> configuration file.
>> > + </para>
>> >
>> > To install the extension itself..
>> > I'm sure we have a standard entity for installing pecl extensions
>> > which would be a lot better.
>> > Also note the wrong indenting of the programlisting.
>> >
>> > The ini section seems to use 2 spaces for indenting rather then the
>> > standard 1space
> 
> And this is now online (the doc mirror), which may reveal a few more details
> like missing return values.
> 
>  - http://docs.php.net/amqp
> 
> Be sure to review each prototype too as docgen defaults to 'void' for skeleton
> documentation return types, and usually 'string' for parameter types. And I
> assume most of your methods should use one of the following within the
> returnvalues role:
> 
>  - &return.success; (bool)
>  - &return.void;    (void)
> 
> But overall this is an example of "How to release a PECL extension, the right
> way." ;)
> 
> Regards,
> Philip

Reply via email to