Re: RFR: (T) 8241144 Javadoc is not generated for new module jdk.nio.mapmode

2020-03-25 Thread Magnus Ihse Bursie
On 2020-03-25 02:22, David Holmes wrote: On 25/03/2020 3:49 am, Florian Weimer wrote: * Magnus Ihse Bursie: On 2020-03-24 09:59, Andrew Dinn wrote: On 23/03/2020 18:38, Erik Joelsson wrote: Looks good. Thanks for the review, Erik. I'm assuming that also implies it is trivial (because,

Re: RFR: (T) 8241144 Javadoc is not generated for new module jdk.nio.mapmode

2020-03-24 Thread David Holmes
On 25/03/2020 3:49 am, Florian Weimer wrote: * Magnus Ihse Bursie: On 2020-03-24 09:59, Andrew Dinn wrote: On 23/03/2020 18:38, Erik Joelsson wrote: Looks good. Thanks for the review, Erik. I'm assuming that also implies it is trivial (because, copyright update a side, it really is a

Re: RFR: (T) 8241144 Javadoc is not generated for new module jdk.nio.mapmode

2020-03-24 Thread Florian Weimer
* Magnus Ihse Bursie: > On 2020-03-24 09:59, Andrew Dinn wrote: >> On 23/03/2020 18:38, Erik Joelsson wrote: >>> Looks good. >> Thanks for the review, Erik. >> >> I'm assuming that also implies it is trivial (because, copyright update >> a side, it really is a 1-liner :-). > > For code in the

Re: RFR: (T) 8241144 Javadoc is not generated for new module jdk.nio.mapmode

2020-03-24 Thread Andrew Dinn
On 24/03/2020 09:10, Magnus Ihse Bursie wrote: > On 2020-03-24 09:59, Andrew Dinn wrote: >> I'm assuming that also implies it is trivial (because, copyright update >> a side, it really is a 1-liner :-). > > For code in the build system, we do not have the Hotspot rules of > multiple reviewers,

Re: RFR: (T) 8241144 Javadoc is not generated for new module jdk.nio.mapmode

2020-03-24 Thread Magnus Ihse Bursie
On 2020-03-24 09:59, Andrew Dinn wrote: On 23/03/2020 18:38, Erik Joelsson wrote: Looks good. Thanks for the review, Erik. I'm assuming that also implies it is trivial (because, copyright update a side, it really is a 1-liner :-). For code in the build system, we do not have the Hotspot

Re: RFR: (T) 8241144 Javadoc is not generated for new module jdk.nio.mapmode

2020-03-24 Thread Andrew Dinn
On 23/03/2020 18:38, Erik Joelsson wrote: > Looks good. Thanks for the review, Erik. I'm assuming that also implies it is trivial (because, copyright update a side, it really is a 1-liner :-). I will push to the dev tree and request a backport to jdk14u. regards, Andrew Dinn ---

Re: RFR: (T) 8241144 Javadoc is not generated for new module jdk.nio.mapmode

2020-03-23 Thread Erik Joelsson
Looks good. /Erik On 2020-03-23 10:56, Andrew Dinn wrote: Could I please have a review for this trivial fix to the module make file which ensures that javadoc is generated for new module jdk.nio.mapmode created as part of the implementation of JEP 352. The original patch added the module to

RFR: (T) 8241144 Javadoc is not generated for new module jdk.nio.mapmode

2020-03-23 Thread Andrew Dinn
Could I please have a review for this trivial fix to the module make file which ensures that javadoc is generated for new module jdk.nio.mapmode created as part of the implementation of JEP 352. The original patch added the module to the BOOT_MODULES list but was not to the DOCS_MODULES list.