On Thu, 28 Mar 2024 08:49:44 GMT, drmarmac wrote:
>> This PR removes potentially incorrect usages of Stream.peek().
>> The changed code should be covered by the tests that are already present.
>
> drmarmac has updated the pull request incrementally with two additional
> commits since the last
On Thu, 28 Mar 2024 22:21:32 GMT, drmarmac wrote:
>> modules/javafx.controls/src/main/java/javafx/scene/control/ControlUtils.java
>> line 166:
>>
>>> 164: sm.startAtomic();
>>> 165:
>>> 166: final List removed = new
>>> ArrayList<>(c.getRemovedSize());
>>
>> I wonder
On Thu, 28 Mar 2024 17:53:14 GMT, Andy Goryachev wrote:
>> drmarmac has updated the pull request incrementally with two additional
>> commits since the last revision:
>>
>> - Preallocate in SelectedIndicesList.set()
>> - Remove unused import
>
>
On Thu, 28 Mar 2024 08:49:44 GMT, drmarmac wrote:
>> This PR removes potentially incorrect usages of Stream.peek().
>> The changed code should be covered by the tests that are already present.
>
> drmarmac has updated the pull request incrementally with two additional
> commits since the last
On Thu, 28 Mar 2024 15:48:50 GMT, Andy Goryachev wrote:
> a minor question: would it be much easier to understand if the code was
> written in conventional procedural style? a straightforward `for` loop?
Even if it were (and I'm not sure it would be), it would be a more intrusive
change than
On Thu, 28 Mar 2024 08:49:44 GMT, drmarmac wrote:
>> This PR removes potentially incorrect usages of Stream.peek().
>> The changed code should be covered by the tests that are already present.
>
> drmarmac has updated the pull request incrementally with two additional
> commits since the last
On Thu, 28 Mar 2024 08:49:44 GMT, drmarmac wrote:
>> This PR removes potentially incorrect usages of Stream.peek().
>> The changed code should be covered by the tests that are already present.
>
> drmarmac has updated the pull request incrementally with two additional
> commits since the last
On Thu, 28 Mar 2024 08:49:44 GMT, drmarmac wrote:
>> This PR removes potentially incorrect usages of Stream.peek().
>> The changed code should be covered by the tests that are already present.
>
> drmarmac has updated the pull request incrementally with two additional
> commits since the last
On Thu, 28 Mar 2024 08:49:44 GMT, drmarmac wrote:
>> This PR removes potentially incorrect usages of Stream.peek().
>> The changed code should be covered by the tests that are already present.
>
> drmarmac has updated the pull request incrementally with two additional
> commits since the last
On Thu, 28 Mar 2024 08:49:44 GMT, drmarmac wrote:
>> This PR removes potentially incorrect usages of Stream.peek().
>> The changed code should be covered by the tests that are already present.
>
> drmarmac has updated the pull request incrementally with two additional
> commits since the last
> This PR removes potentially incorrect usages of Stream.peek().
> The changed code should be covered by the tests that are already present.
drmarmac has updated the pull request incrementally with two additional commits
since the last revision:
- Preallocate in SelectedIndicesList.set()
-
11 matches
Mail list logo
