Re: RFR: 8273616: Fix trivial doc typos in the java.base module [v3]

2021-09-13 Thread John R Rose
On Mon, 13 Sep 2021 11:22:27 GMT, Pavel Rappo wrote: >> 8273616: Fix trivial doc typos in the java.base module > > Pavel Rappo has updated the pull request incrementally with one additional > commit since the last revision: > > Use "ensure" instead of "insure" Marked as reviewed by jrose (Re

Re: RFR: 8273616: Fix trivial doc typos in the java.base module [v3]

2021-09-13 Thread Lance Andersen
On Mon, 13 Sep 2021 11:22:27 GMT, Pavel Rappo wrote: >> 8273616: Fix trivial doc typos in the java.base module > > Pavel Rappo has updated the pull request incrementally with one additional > commit since the last revision: > > Use "ensure" instead of "insure" Marked as reviewed by lancea (R

Re: RFR: 8273616: Fix trivial doc typos in the java.base module [v3]

2021-09-13 Thread Iris Clark
On Mon, 13 Sep 2021 11:22:27 GMT, Pavel Rappo wrote: >> 8273616: Fix trivial doc typos in the java.base module > > Pavel Rappo has updated the pull request incrementally with one additional > commit since the last revision: > > Use "ensure" instead of "insure" Marked as reviewed by iris (Rev

Re: RFR: 8273616: Fix trivial doc typos in the java.base module [v3]

2021-09-13 Thread Roger Riggs
On Mon, 13 Sep 2021 11:22:27 GMT, Pavel Rappo wrote: >> 8273616: Fix trivial doc typos in the java.base module > > Pavel Rappo has updated the pull request incrementally with one additional > commit since the last revision: > > Use "ensure" instead of "insure" Marked as reviewed by rriggs (R

Re: RFR: 8273616: Fix trivial doc typos in the java.base module [v3]

2021-09-13 Thread Daniel Fuchs
On Mon, 13 Sep 2021 11:22:27 GMT, Pavel Rappo wrote: >> 8273616: Fix trivial doc typos in the java.base module > > Pavel Rappo has updated the pull request incrementally with one additional > commit since the last revision: > > Use "ensure" instead of "insure" LGTM - Marked as

Re: RFR: 8273616: Fix trivial doc typos in the java.base module [v3]

2021-09-13 Thread Pavel Rappo
> 8273616: Fix trivial doc typos in the java.base module Pavel Rappo has updated the pull request incrementally with one additional commit since the last revision: Use "ensure" instead of "insure" - Changes: - all: https://git.openjdk.java.net/jdk/pull/5475/files - new: https

Re: RFR: 8273616: Fix trivial doc typos in the java.base module [v2]

2021-09-13 Thread Pavel Rappo
On Fri, 10 Sep 2021 23:20:11 GMT, Pavel Rappo wrote: >> 8273616: Fix trivial doc typos in the java.base module > > Pavel Rappo has updated the pull request incrementally with one additional > commit since the last revision: > > Revert two fixes Thanks to those who reviewed this PR. Since I p

Re: RFR: 8273616: Fix trivial doc typos in the java.base module [v2]

2021-09-11 Thread Lance Andersen
On Fri, 10 Sep 2021 23:20:11 GMT, Pavel Rappo wrote: >> 8273616: Fix trivial doc typos in the java.base module > > Pavel Rappo has updated the pull request incrementally with one additional > commit since the last revision: > > Revert two fixes Marked as reviewed by lancea (Reviewer). -

Re: RFR: 8273616: Fix trivial doc typos in the java.base module [v2]

2021-09-10 Thread Iris Clark
On Fri, 10 Sep 2021 23:20:11 GMT, Pavel Rappo wrote: >> 8273616: Fix trivial doc typos in the java.base module > > Pavel Rappo has updated the pull request incrementally with one additional > commit since the last revision: > > Revert two fixes Marked as reviewed by iris (Reviewer). ---

Re: RFR: 8273616: Fix trivial doc typos in the java.base module [v2]

2021-09-10 Thread Pavel Rappo
> 8273616: Fix trivial doc typos in the java.base module Pavel Rappo has updated the pull request incrementally with one additional commit since the last revision: Revert two fixes - Changes: - all: https://git.openjdk.java.net/jdk/pull/5475/files - new: https://git.openjdk.j

Re: RFR: 8273616: Fix trivial doc typos in the java.base module [v2]

2021-09-10 Thread Pavel Rappo
On Fri, 10 Sep 2021 21:52:36 GMT, John R Rose wrote: >> Pavel Rappo has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Revert two fixes > > src/java.base/share/classes/java/nio/channels/FileChannel.java line 567: > >> 565: *

Re: RFR: 8273616: Fix trivial doc typos in the java.base module

2021-09-10 Thread Florent Guillaume
On Fri, 10 Sep 2021 21:51:45 GMT, John R Rose wrote: >> 8273616: Fix trivial doc typos in the java.base module > > src/java.base/share/classes/java/nio/channels/AsynchronousFileChannel.java > line 399: > >> 397: * If {@code true} then this method is required to force >> changes >

Re: RFR: 8273616: Fix trivial doc typos in the java.base module

2021-09-10 Thread John R Rose
On Fri, 10 Sep 2021 21:16:19 GMT, Pavel Rappo wrote: > 8273616: Fix trivial doc typos in the java.base module Approved, except for two changes commented above: the original "it need only force" is correct usage, and "it needs only force" is not good usage, but "it only needs to force" would be

RFR: 8273616: Fix trivial doc typos in the java.base module

2021-09-10 Thread Pavel Rappo
8273616: Fix trivial doc typos in the java.base module - Commit messages: - Initial commit Changes: https://git.openjdk.java.net/jdk/pull/5475/files Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=5475&range=00 Issue: https://bugs.openjdk.java.net/browse/JDK-8273616 Stats