On Tue, Jun 5, 2012 at 8:30 PM, Hannes Magnusson
<hannes.magnus...@gmail.com> wrote:
> On Tue, Jun 5, 2012 at 8:04 PM, Moacir de Oliveira
> <moacirdeoliveira....@gmail.com> wrote:
>> Hi,
>>
>> 2012/6/5 Hannes Magnusson <hannes.magnus...@gmail.com>
>>>
>>>
>>> Attached is a patch that introduces phd:argN attribute to
>>> <function/methodname> and will cause <methodname phd:arg1="PDO::bar"
>>> phd:arg2="PDO::baz>PDOStatement::foo</methodname> to be rendered as:
>>> <span..><a hre...>PDOStatement::foo(PDO::bar, PDO::baz)</a></span>
>>> That is, it will make the phd:argN look like proper arguments to the
>>> function.
>>>
>>> Should we include include this in the coming PhD release before Friday?
>>>
>>> -Hannes
>>
>>
>> Oops... now the correct thread with the correct patch :D.
>>
>> So my suggestion is to use phd:args="arg1, arg2, arg3...", so we will not
>> have a limited number of arguments in the dtd specification.
>
> I guess that makes sense. I was wondering if we could do any
> additional processing of the arguments such as linking constants or
> whatever, but that doesn't make whole lot of sense.


Committed.

-Hannes

Reply via email to