On 9 March 2012 10:50, Benedikt Ritter <benerit...@googlemail.com> wrote:
> Thanks for the patch Dennis! Feel free create a JIRA ticket [1] for
> this issue, where you can attach your patch (and maybe more ;). One
> little suggestion for improvement: According to [2] {@code } should be
> used in instead of <code> and <tt>.

<code> still needs to be used in some cases, because {@code} does not
support some types of nested content.
[Cannot remember exactly what, sorry, but I had to change some @code tags back.]

> Regards,
> Benedikt
>
> [1] https://issues.apache.org/jira
> [2] https://blogs.oracle.com/darcy/entry/javadoc_tip_code_and_literal
>
> Am 9. März 2012 11:34 schrieb Dennis Hendriks <d.hendr...@tue.nl>:
>> Hi all,
>>
>> See http://commons.apache.org/math/apidocs/index-all.html and scroll to the
>> section 'D'. Observe how in method discardMostRecentElements(int) in class
>> org.apache.commons.math3.util.ResizableDoubleArray, the style changes to
>> larger font. This can be fixed using this patch:
>>
>>
>> Index: src/main/java/org/apache/commons/math3/util/ResizableDoubleArray.java
>> ===================================================================
>> --- src/main/java/org/apache/commons/math3/util/ResizableDoubleArray.java
>>    (revision 1298781)
>> +++ src/main/java/org/apache/commons/math3/util/ResizableDoubleArray.java
>>    (working copy)
>> @@ -457,7 +457,7 @@
>>     }
>>
>>     /**
>> -     * Discards the <code>i<code> last elements of the array.  For example,
>> +     * Discards the <code>i</code> last elements of the array.  For
>> example,
>>      * if the array contains the elements 1,2,3,4, invoking
>>      * <code>discardMostRecentElements(2)</code> will cause the last two
>> elements
>>      * to be discarded, leaving 1,2 in the array.  Throws
>> illegalArgumentException
>>
>>
>> and is obviously due to <code>i</code> not being balanced. Similar issues
>> exist elsewhere in the javadoc, for instance at the isSame(Chromosome)
>> method in the org.apache.commons.math3.genetics.BinaryChromosome class.
>>
>> Best regards,
>> Dennis
>>
>> ---------------------------------------------------------------------
>> 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