Re: RFR: JDK-8263684: Avoid wrapping into BufferedWriter twice [v3]

2021-05-19 Thread Jonathan Gibbons
> Please review a trivial fix to avoid creating an unnecessary `BufferedWriter`. > > `noreg-trivial` (or `noreg-cleanup` : take your pick). Either way, no test. Jonathan Gibbons has updated the pull request incrementally with one additional commit since the last revision: update copyright yea

Re: RFR: JDK-8263684: Avoid wrapping into BufferedWriter twice [v2]

2021-05-19 Thread Pavel Rappo
On Wed, 19 May 2021 00:00:15 GMT, Jonathan Gibbons wrote: >> Please review a trivial fix to avoid creating an unnecessary >> `BufferedWriter`. >> >> `noreg-trivial` (or `noreg-cleanup` : take your pick). Either way, no test. > > Jonathan Gibbons has updated the pull request incrementally with o

Re: RFR: JDK-8263684: Avoid wrapping into BufferedWriter twice [v2]

2021-05-18 Thread Jonathan Gibbons
> Please review a trivial fix to avoid creating an unnecessary `BufferedWriter`. > > `noreg-trivial` (or `noreg-cleanup` : take your pick). Either way, no test. Jonathan Gibbons has updated the pull request incrementally with one additional commit since the last revision: address review feedb

Re: RFR: JDK-8263684: Avoid wrapping into BufferedWriter twice

2021-05-18 Thread Jonathan Gibbons
On Tue, 18 May 2021 11:00:19 GMT, Pavel Rappo wrote: >> Please review a trivial fix to avoid creating an unnecessary >> `BufferedWriter`. >> >> `noreg-trivial` (or `noreg-cleanup` : take your pick). Either way, no test. > > src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/toolkit/util

Re: RFR: JDK-8263684: Avoid wrapping into BufferedWriter twice

2021-05-18 Thread Pavel Rappo
On Tue, 18 May 2021 04:33:52 GMT, Jonathan Gibbons wrote: > Please review a trivial fix to avoid creating an unnecessary `BufferedWriter`. > > `noreg-trivial` (or `noreg-cleanup` : take your pick). Either way, no test. Changes requested by prappo (Reviewer). src/jdk.javadoc/share/classes/jdk/j

RFR: JDK-8263684: Avoid wrapping into BufferedWriter twice

2021-05-17 Thread Jonathan Gibbons
Please review a trivial fix to avoid creating an unnecessary `BufferedWriter`. `noreg-trivial` (or `noreg-cleanup` : take your pick). Either way, no test. - Commit messages: - JDK-8263684: Avoid wrapping into BufferedWriter twice Changes: https://git.openjdk.java.net/jdk/pull/4080/