RFR: 8288882: JFileChooser - empty (0 bytes) file is displayed as 1 KB [v16]

[Alexey Ivanov]

On Wed, 17 Aug 2022 04:25:26 GMT, Abhishek Kumar <duke at openjdk.org> 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