On Wed, 17 Aug 2022 04:25:26 GMT, Abhishek Kumar <[email protected]> wrote:
> } else if (len < 1024L) { ... } has been removed in previous commit. So if
> listViewWindowsStyle is false then the next else if will be executed i.e.
> <100L.
For clarity, would it be better to separate the cases?
if (listViewWindowsStyle) {
if (len == 0) {
} else {
if (len < 100L) {
}
This would increase the indentation level, yet it would make the code clearer.
Now two independent conditions are chained.
-------------
PR: https://git.openjdk.org/jdk/pull/9327
