RFR: 8255729: com.sun.tools.javac.processing.JavacFiler.FilerOutputStream is inefficient [v2]
github.com+13688759+lgxbslgx at openjdk.java.net
Tue Dec 22 05:40:56 UTC 2020
On Mon, 21 Dec 2020 09:34:59 GMT, Joel Borggrén-Franck <jfranck at openjdk.org> wrote:
>> Guoxiong Li has updated the pull request incrementally with one additional commit since the last revision:
>> Use Objects::checkFromIndexSize
> I'd like to see some kind of evidence that performance was actually improved. I realise this might be hard to make a good standalone test for, but it should be doable as an experiment observing how many times write(int) is called for a huge file.
I revised the code by using `Objects::checkFromIndexSize`. I don't revise the code in `FilterOutputStream` because it is the content of package `java.base/java.io`. If you thought the `FilterOutputStream::write(byte b, int off, int len)` is needed to be modified, I will revise it, too.
Thank you for taking the time to review.
More information about the compiler-dev