I don't see any value in test Javadocs and IMO they should be removed.

Niall


On Sun, Apr 13, 2014 at 10:47 PM, sebb <seb...@gmail.com> wrote:

> On 13 April 2014 18:38, Phil Steitz <phil.ste...@gmail.com> wrote:
> > On 4/13/14, 10:33 AM, sebb wrote:
> >> Agreed; <name> will cause Java 8 Javadoc to report errors.
> >>
> >> If you don't want to use @code, one can always use {name} or [name]
> instead.
> >> Just don't use HTML tag delimiters ...
> >
> > Makes no difference to me, as we don't generate html javadoc for
> > test code.
>
> Actually, the site build does currently include test Javadocs, for example:
>
> http://commons.apache.org/proper/commons-dbcp/testapidocs/index.html
>
> > Change it however you want.
>
> OK
>
> > What would be *really*
> > useful is to help actually resolve the bug that this commit
> > illustrates - it is a material bug in DBCP 2.
> >
> > Phil
> >>
> >> On 13 April 2014 18:21, Gary Gregory <garydgreg...@gmail.com> wrote:
> >>> -1 to using <> instead of {@code}
> >>>
> >>> Gary
> >>>
> >>> <div>-------- Original message --------</div><div>From: Bernd <
> e...@zusammenkunft.net> </div><div>Date:04/13/2014  10:50  (GMT-05:00)
> </div><div>To: Commons Developers List <dev@commons.apache.org>
> </div><div>Subject: Re: svn commit: r1586926 - in
> /commons/proper/dbcp/trunk/src/test/java/org/apache/commons/dbcp2:
> TestConnectionPool.java TestPStmtPoolingBasicDataSource.java </div><div>
> >>> </div>Hello,
> >>>
> >>> Am 13.04.2014 08:11 schrieb <pste...@apache.org>:
> >>>> Author: psteitz
> >>>> Date: Sun Apr 13 06:10:44 2014
> >>>> New Revision: 1586926
> >>>>
> >>>> URL: http://svn.apache.org/r1586926
> >>>> Log:
> >>>> Added test (commented out) to illustrate DBCP-415.
> >>>>
> >>>> Modified:
> >>>>
> >>>
> commons/proper/dbcp/trunk/src/test/java/org/apache/commons/dbcp2/TestConnectionPool.java
> >>>
> commons/proper/dbcp/trunk/src/test/java/org/apache/commons/dbcp2/TestPStmtPoolingBasicDataSource.java
> >>>> Modified:
> >>>
> commons/proper/dbcp/trunk/src/test/java/org/apache/commons/dbcp2/TestConnectionPool.java
> >>>> URL:
> >>>
> http://svn.apache.org/viewvc/commons/proper/dbcp/trunk/src/test/java/org/apache/commons/dbcp2/TestConnectionPool.java?rev=1586926&r1=1586925&r2=1586926&view=diff
> >>>
> ==============================================================================
> >>>> ---
> >>>
> commons/proper/dbcp/trunk/src/test/java/org/apache/commons/dbcp2/TestConnectionPool.java
> >>> (original)
> >>>> +++
> >>>
> commons/proper/dbcp/trunk/src/test/java/org/apache/commons/dbcp2/TestConnectionPool.java
> >>> Sun Apr 13 06:10:44 2014
> >>> ...
> >>>> +    /**
> >>>> +     * Launches a group of <numThreads> threads, each of which will
> >>> attempt to obtain a connection
> >>>
> >>> Is using <name> a good idea in Javadoc? I think officially you would
> need
> >>> to use @{code name}.
> >>>
> >>> ...
> >> ---------------------------------------------------------------------
> >> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> >> For additional commands, e-mail: dev-h...@commons.apache.org
> >>
> >>
> >
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> > For additional commands, e-mail: dev-h...@commons.apache.org
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
> For additional commands, e-mail: dev-h...@commons.apache.org
>
>

Reply via email to