On Tue, 16 Aug 2022 19:21:12 GMT, Phil Race <p...@openjdk.org> wrote:

>> It's reachable when `listViewWindowsStyle` is `false`, which seems to be the 
>> case in all the Look-and-Feels but Windows one.
>> 
>> Yet it looks confusing.
>
>> It's reachable when `listViewWindowsStyle` is `false`, which seems to be the 
>> case in all the Look-and-Feels but Windows one.
>> 
>> Yet it looks confusing.
> 
> We have
>                 if (listViewWindowsStyle) {
>                      ..
>                  } else if (len < 1024L) {
>                      ...
>                  } else if (len < 100L) {
> 
> So if listViewWindowsStyle is false then we get to < 1024 case and if there 
> len is (say) 20
> we enter that, so as far as I can see we can never reach the < 100 case.

} else if (len < 1024L) {
...
} has been removed. 
So if listViewWindowStyle is false then the next else if condition will be 
executed i.e. <100L case.

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

PR: https://git.openjdk.org/jdk/pull/9327

Reply via email to