On Tue, 19 Jul 2022 05:49:20 GMT, Abhishek Kumar <[email protected]> wrote:

>> I think this also deserves extracting it into a separate function, and a 
>> unit test.
>> And perhaps another unit test with a non-English locale.
>
>> So ALL of these have an "off by one" issue don't they ?
>> 
>> Here a file that is (for example) 2047 bytes in size will be displayed as 
>> "1KB"
> 
> There is "off by one" issue in this case. One possible solution can be - 
> 
> 
> ![FileChooser_FileSizes](https://user-images.githubusercontent.com/107542245/179674833-345a7e81-4af2-403e-b9e1-9d690bffeddd.png)

If any file having size  >1024 byte & <2048 byte will be rounded off as 2 KB 
and so on.
Similarly if file size is >1023KB will show file size as 1MB (rounding off will 
make file size as 1024KB and then it will convert it to MB).

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

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

Reply via email to