Hi Peter:

On Wed, May 05, 2010 at 09:02:57AM +0100, Peter Cowburn wrote:
> On 5 May 2010 01:51, Daniel Convissor <dani...@php.net> wrote:
> > danielc ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ? ?Wed, 05 May 2010 00:51:18 +0000
> >
> > Revision: http://svn.php.net/viewvc?view=revision&revision=299003
> >
> > Log:
> > Use function instead of methodname in seealso to get () in output.
> 
> Please use the most appropriate tag for its content, in this case
> methodname. If you want method names to have parentheses suffixed,
> raise a bug.

I KNEW this commit was going to spark a conversation. :)  Raising a bug 
about this is on my to do list.  Thanks for the round tuit...
http://bugs.php.net/bug.php?id=51750

I went with <function> for now because everything else in the section was 
using it already, it's not a big deal, and so I just went with the 
flow... only to be slammed against a rock. :)

As mentioned in the bug report, I am guessing the parentheses are not 
added currently because we don't want them added inside <methodsynopsis> 
-- meaning "Foo::Bar() (string $string)" would look silly.  So PhD needs 
to add "()" only when not in a methodsynopsis.

Thanks,

--Dan

-- 
 T H E   A N A L Y S I S   A N D   S O L U T I O N S   C O M P A N Y
            data intensive web and database programming
                http://www.AnalysisAndSolutions.com/
 4015 7th Ave #4, Brooklyn NY 11232  v: 718-854-0335 f: 718-854-0409

Reply via email to