Re: RFR 8148617: top level make docs target does not generate javadocs for dynalink API

2016-01-29 Thread Hannes Wallnoefer
+1 Am 2016-01-29 um 18:11 schrieb Sundararajan Athijegannathan: Added newline before line 1270 in Javadoc.gmk. Updated webrev: http://cr.openjdk.java.net/~sundar/8148617/webrev.01/ -Sundar On 1/29/2016 9:53 PM, Hannes Wallnoefer wrote: Looks good. Hannes Am 2016-01-29 um 17:13 schrieb Sun

Re: RFR 8148617: top level make docs target does not generate javadocs for dynalink API

2016-01-29 Thread Erik Joelsson
Looks good. Erik On 2016-01-29 18:11, Sundararajan Athijegannathan wrote: Added newline before line 1270 in Javadoc.gmk. Updated webrev: http://cr.openjdk.java.net/~sundar/8148617/webrev.01/ -Sundar On 1/29/2016 9:53 PM, Hannes Wallnoefer wrote: Looks good. Hannes Am 2016-01-29 um 17:13

Re: RFR 8148617: top level make docs target does not generate javadocs for dynalink API

2016-01-29 Thread Sundararajan Athijegannathan
Added newline before line 1270 in Javadoc.gmk. Updated webrev: http://cr.openjdk.java.net/~sundar/8148617/webrev.01/ -Sundar On 1/29/2016 9:53 PM, Hannes Wallnoefer wrote: Looks good. Hannes Am 2016-01-29 um 17:13 schrieb Sundararajan Athijegannathan: Please review http://cr.openjdk.java.ne

Re: RFR 8148617: top level make docs target does not generate javadocs for dynalink API

2016-01-29 Thread Hannes Wallnoefer
Looks good. Hannes Am 2016-01-29 um 17:13 schrieb Sundararajan Athijegannathan: Please review http://cr.openjdk.java.net/~sundar/8148617/webrev.00/ for https://bugs.openjdk.java.net/browse/JDK-8148617 Dynalink API is specified by JEP-276 (http://openjdk.java.net/jeps/276). The current chang

Re: RFR 8148617: top level make docs target does not generate javadocs for dynalink API

2016-01-29 Thread Erik Joelsson
Looks good. /Erik On 2016-01-29 17:13, Sundararajan Athijegannathan wrote: Please review http://cr.openjdk.java.net/~sundar/8148617/webrev.00/ for https://bugs.openjdk.java.net/browse/JDK-8148617 Dynalink API is specified by JEP-276 (http://openjdk.java.net/jeps/276). The current change is j