Re: RFR: 8271865: SortedList::getViewIndex behaves not correctly for some index values [v2]

[drmarmac]

On Thu, 2 May 2024 12:18:30 GMT, Kevin Rushforth <k...@openjdk.org> wrote:

>> drmarmac has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   JavaDoc updates
>
> modules/javafx.base/src/main/java/javafx/collections/transformation/TransformationList.java
>  line 121:
> 
>> 119:      * @param index the index in this list
>> 120:      * @return the index of the element's origin in the source list
>> 121:      * @throws IndexOutOfBoundsException if the index is out of range 
>> (<code>index &lt; 0 || index >= size()</code>)
> 
> In javadoc comments, we prefer using `{@code ... }` rather than the raw HTML 
> tags. This also allows using `<` rather than `&lt;`.
> 
> Suggested change:
> 
> 
> ... ({@code index < 0 || index >= size()})

ok, looks better!

-------------

PR Review Comment: https://git.openjdk.org/jfx/pull/1432#discussion_r1587609916