On Sat, 13 Mar 2021 14:40:14 GMT, Philippe Marschall <github.com+471021+marsch...@openjdk.org> wrote:
>> I think the implementation changes here look good. I don't know however >> whether there is enough coverage in the tests. These should verify that the >> `Reader`, `CharArrayReader`, and `InputStreamReader` implementations of >> `read(CharBuffer)` are accurate. If there is already sufficient coverage in >> the tests in `test/jdk/java/io` then that is good enough and nothing need be >> added. > >> I think the implementation changes here look good. I don't know however >> whether there is enough coverage in the tests. These should verify that the >> `Reader`, `CharArrayReader`, and `InputStreamReader` implementations of >> `read(CharBuffer)` are accurate. If there is already sufficient coverage in >> the tests in `test/jdk/java/io` then that is good enough and nothing need be >> added. > > `CharArrayReader` was lacking a test. I added a test which found a bug and > fixed the bug. The PR also contains new tests for `Reader` and > `InputStreamReader`. They cover on-heap and off-heap cases. > > Is there a way to get test coverage with JTReg tests? I only found [1] which > seems out of date and points to an Oracle internal wiki. > > [1] > https://wiki.openjdk.java.net/display/CodeTools/JCov+FAQ#JCovFAQ-HowdoIuseJCovwithjtreg? How do we proceed here? Are there additional changes that you would like me to perform or undo? ------------- PR: https://git.openjdk.java.net/jdk/pull/1915