# HG changeset patch # User bpatel # Date 1506556027 25200 # Node ID 355349babaf4d26c715d19eaf163c3d4b8ffa9d8 # Parent ce5fd3ba3fea1c4072ccb3d11585e2a154232587 8186332: Fix method signature in method summary table Reviewed-by: jjg, ksrini diff -r ce5fd3ba3fea -r 355349babaf4 src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/AbstractExecutableMemberWriter.java --- a/src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/AbstractExecutableMemberWriter.java Wed Sep 27 14:23:41 2017 -0700 +++ b/src/jdk.javadoc/share/classes/jdk/javadoc/internal/doclets/formats/html/AbstractExecutableMemberWriter.java Wed Sep 27 16:47:07 2017 -0700 @@ -39,6 +39,7 @@ import javax.lang.model.type.TypeVariable; import javax.lang.model.util.SimpleTypeVisitor9; +import jdk.javadoc.internal.doclets.formats.html.markup.ContentBuilder; import jdk.javadoc.internal.doclets.formats.html.markup.HtmlStyle; import jdk.javadoc.internal.doclets.formats.html.markup.HtmlTree; import jdk.javadoc.internal.doclets.toolkit.Content; @@ -199,55 +200,61 @@ */ protected void addParameters(ExecutableElement member, boolean includeAnnotations, Content htmltree, int indentSize) { - htmltree.addContent(Contents.ZERO_WIDTH_SPACE); - htmltree.addContent("("); + Content paramTree = new ContentBuilder(); String sep = ""; List parameters = member.getParameters(); CharSequence indent = makeSpace(indentSize + 1); TypeMirror rcvrType = member.getReceiverType(); if (includeAnnotations && rcvrType != null && utils.isAnnotated(rcvrType)) { List annotationMirrors = rcvrType.getAnnotationMirrors(); - addReceiverAnnotations(member, rcvrType, annotationMirrors, htmltree); + addReceiverAnnotations(member, rcvrType, annotationMirrors, paramTree); sep = "," + DocletConstants.NL + indent; } int paramstart; for (paramstart = 0; paramstart < parameters.size(); paramstart++) { - htmltree.addContent(sep); + paramTree.addContent(sep); VariableElement param = parameters.get(paramstart); if (param.getKind() != ElementKind.INSTANCE_INIT) { if (includeAnnotations) { boolean foundAnnotations = writer.addAnnotationInfo(indent.length(), - member, param, htmltree); + member, param, paramTree); if (foundAnnotations) { - htmltree.addContent(DocletConstants.NL); - htmltree.addContent(indent); + paramTree.addContent(DocletConstants.NL); + paramTree.addContent(indent); } } addParam(member, param, - (paramstart == parameters.size() - 1) && member.isVarArgs(), htmltree); + (paramstart == parameters.size() - 1) && member.isVarArgs(), paramTree); break; } } for (int i = paramstart + 1; i < parameters.size(); i++) { - htmltree.addContent(","); - htmltree.addContent(DocletConstants.NL); - htmltree.addContent(indent); + paramTree.addContent(","); + paramTree.addContent(DocletConstants.NL); + paramTree.addContent(indent); if (includeAnnotations) { boolean foundAnnotations = writer.addAnnotationInfo(indent.length(), member, parameters.get(i), - htmltree); + paramTree); if (foundAnnotations) { - htmltree.addContent(DocletConstants.NL); - htmltree.addContent(indent); + paramTree.addContent(DocletConstants.NL); + paramTree.addContent(indent); } } addParam(member, parameters.get(i), (i == parameters.size() - 1) && member.isVarArgs(), - htmltree); + paramTree); } - htmltree.addContent(")"); + if (paramTree.isEmpty()) { + htmltree.addContent("()"); + } else { + htmltree.addContent(Contents.ZERO_WIDTH_SPACE); + htmltree.addContent("("); + htmltree.addContent(paramTree); + paramTree.addContent(")"); + } } /** diff -r ce5fd3ba3fea -r 355349babaf4 test/langtools/jdk/javadoc/doclet/testClassLinks/TestClassLinks.java --- a/test/langtools/jdk/javadoc/doclet/testClassLinks/TestClassLinks.java Wed Sep 27 14:23:41 2017 -0700 +++ b/test/langtools/jdk/javadoc/doclet/testClassLinks/TestClassLinks.java Wed Sep 27 16:47:07 2017 -0700 @@ -23,7 +23,7 @@ /* * @test - * @bug 8163800 8175200 + * @bug 8163800 8175200 8186332 * @summary The fix for JDK-8072052 shows up other minor incorrect use of styles * @library ../lib * @modules jdk.javadoc/jdk.javadoc.internal.tool @@ -51,11 +51,11 @@ checkOutput("p/C1.html", true, "C2", - "C1​()"); + "C1()"); checkOutput("p/C2.html", true, "C3", - "C2​()"); + "C2()"); checkOutput("p/C3.html", true, "I1, " @@ -63,7 +63,7 @@ + "I2, " + "IT1<T>, " + "IT2<java.lang.String>", - "C3​()"); + "C3()"); checkOutput("p/I1.html", true, "C3", diff -r ce5fd3ba3fea -r 355349babaf4 test/langtools/jdk/javadoc/doclet/testConstructors/TestConstructors.java --- a/test/langtools/jdk/javadoc/doclet/testConstructors/TestConstructors.java Wed Sep 27 14:23:41 2017 -0700 +++ b/test/langtools/jdk/javadoc/doclet/testConstructors/TestConstructors.java Wed Sep 27 16:47:07 2017 -0700 @@ -23,7 +23,7 @@ /* * @test - * @bug 8025524 8031625 8081854 8175200 + * @bug 8025524 8031625 8081854 8175200 8186332 * @summary Test for constructor name which should be a non-qualified name. * @author Bhavesh Patel * @library ../lib @@ -58,19 +58,19 @@ + "Outer(int), " + "" + "NestedInner(int)", - "Outer​()", + "Outer()", "", "Outer​(int i)", ""); checkOutput("pkg1/Outer.Inner.html", true, - "Inner​()", + "Inner()", "", "Inner​(int i)", ""); checkOutput("pkg1/Outer.Inner.NestedInner.html", true, - "NestedInner​()", + "NestedInner()", "", "NestedInner​(int i)", ""); diff -r ce5fd3ba3fea -r 355349babaf4 test/langtools/jdk/javadoc/doclet/testDeprecatedDocs/TestDeprecatedDocs.java --- a/test/langtools/jdk/javadoc/doclet/testDeprecatedDocs/TestDeprecatedDocs.java Wed Sep 27 14:23:41 2017 -0700 +++ b/test/langtools/jdk/javadoc/doclet/testDeprecatedDocs/TestDeprecatedDocs.java Wed Sep 27 16:47:07 2017 -0700 @@ -23,7 +23,7 @@ /* * @test - * @bug 4927552 8026567 8071982 8162674 8175200 8175218 8183511 + * @bug 4927552 8026567 8071982 8162674 8175200 8175218 8183511 8186332 * @summary * @author jamieh * @library ../lib @@ -83,10 +83,10 @@ + "public int field\n" + "
Deprecated, for removal: This API element is subject to removal in a future version. 
", "
@Deprecated(forRemoval=true)\n"
-                + "public DeprecatedClassByAnnotation​()
\n" + + "public DeprecatedClassByAnnotation()\n" + "
Deprecated, for removal: This API element is subject to removal in a future version. 
", "
@Deprecated\n"
-                + "public void method​()
\n" + + "public void method()\n" + "
Deprecated. 
"); checkOutput("pkg/TestAnnotationType.html", true, @@ -117,7 +117,7 @@ + "
class_test1 passes.
\n" + "", "
@Deprecated(forRemoval=true)\n"
-                + "public TestClass​()
\n" + + "public TestClass()\n" + "
Deprecated, for removal: This API element is subject to removal in a future version. " + "class_test3 passes.
"); diff -r ce5fd3ba3fea -r 355349babaf4 test/langtools/jdk/javadoc/doclet/testHtmlDefinitionListTag/TestHtmlDefinitionListTag.java --- a/test/langtools/jdk/javadoc/doclet/testHtmlDefinitionListTag/TestHtmlDefinitionListTag.java Wed Sep 27 14:23:41 2017 -0700 +++ b/test/langtools/jdk/javadoc/doclet/testHtmlDefinitionListTag/TestHtmlDefinitionListTag.java Wed Sep 27 16:47:07 2017 -0700 @@ -23,7 +23,7 @@ /* * @test - * @bug 6786690 6820360 8025633 8026567 8175200 8183511 + * @bug 6786690 6820360 8025633 8026567 8175200 8183511 8186332 * @summary This test verifies the nesting of definition list tags. * @author Bhavesh Patel * @library ../lib @@ -367,12 +367,12 @@ // Test with -nocomment and -nodeprecated options. The ClassDocs whould // not display definition lists for any member details. checkOutput("pkg1/C1.html", expectFound, - "
public void readObject​()\n" +
+                "
public void readObject()\n" +
                 "                throws java.io.IOException
\n" + ""); checkOutput("pkg1/C2.html", expectFound, - "
public C2​()
\n" + + "
public C2()
\n" + ""); checkOutput("pkg1/C1.ModalExclusionType.html", expectFound, diff -r ce5fd3ba3fea -r 355349babaf4 test/langtools/jdk/javadoc/doclet/testInterface/TestInterface.java --- a/test/langtools/jdk/javadoc/doclet/testInterface/TestInterface.java Wed Sep 27 14:23:41 2017 -0700 +++ b/test/langtools/jdk/javadoc/doclet/testInterface/TestInterface.java Wed Sep 27 16:47:07 2017 -0700 @@ -23,7 +23,7 @@ /* * @test - * @bug 4682448 4947464 5029946 8025633 8026567 8035473 8139101 8175200 + * @bug 4682448 4947464 5029946 8025633 8026567 8035473 8139101 8175200 8186332 * @summary Verify that the public modifier does not show up in the * documentation for public methods, as recommended by the JLS. * If A implements I and B extends A, B should be in the list of @@ -64,7 +64,7 @@ checkExit(Exit.OK); checkOutput("pkg/Interface.html", true, - "
int method​()
", + "
int method()
", "
static final int field
", // Make sure known implementing class list is correct and omits type parameters. "
\n" @@ -119,7 +119,7 @@ + "
"); checkOutput("pkg/Interface.html", false, - "public int method​()", + "public int method()", "public static final int field"); checkOutput("pkg/ClassWithStaticMethod.html", false, diff -r ce5fd3ba3fea -r 355349babaf4 test/langtools/jdk/javadoc/doclet/testJavaFX/TestJavaFX.java --- a/test/langtools/jdk/javadoc/doclet/testJavaFX/TestJavaFX.java Wed Sep 27 14:23:41 2017 -0700 +++ b/test/langtools/jdk/javadoc/doclet/testJavaFX/TestJavaFX.java Wed Sep 27 16:47:07 2017 -0700 @@ -24,7 +24,7 @@ /* * @test * @bug 7112427 8012295 8025633 8026567 8061305 8081854 8150130 8162363 - * 8167967 8172528 8175200 8178830 8182257 + * 8167967 8172528 8175200 8178830 8182257 8186332 * @summary Test of the JavaFX doclet features. * @author jvalenta * @library ../lib @@ -58,7 +58,7 @@ + "
Sets the value of the property rate.
\n" + "
\n" + "
Property description:
", - "
public final double getRate​()
\n" + "
public final double getRate()
\n" + "
Gets the value of the property rate.
\n" + "
\n" + "
Property description:
", @@ -78,7 +78,7 @@ "Property description:", "" + "
" - + "setTestMethodProperty​()", + + "setTestMethodProperty()", "" + "paused\n" + "\n" @@ -88,7 +88,7 @@ + "title=\"class in pkg1\">C.BooleanProperty pausedProperty
\n" + "
Defines if paused. The second line.
", "

isPaused

\n" - + "
public final double isPaused​()
\n" + + "
public final double isPaused()
\n" + "
Gets the value of the property paused.
", "

setPaused

\n" + "
public final void setPaused​(boolean value)
\n" @@ -99,7 +99,7 @@ + "
Default value:
\n" + "
false
", "

isPaused

\n" - + "
public final double isPaused​()
\n" + + "
public final double isPaused()
\n" + "
Gets the value of the property paused.
\n" + "
\n" + "
Property description:
\n" @@ -124,7 +124,7 @@ + "
Since:
\n" + "
JavaFX 8.0
", "

getRate

\n" - + "
public final double getRate​()
\n" + + "
public final double getRate()
\n" + "
Gets the value of the property rate.
\n" + "
\n" + "
Property description:
\n" @@ -253,20 +253,20 @@ + "\n" + "java.lang.Object\n" + "" - + "betaProperty​()\n" + + "betaProperty()\n" + " \n" + "\n" + "\n" + "java.util.List<java.util.Set<? super java.lang.Object>>" + "\n" + "" - + "deltaProperty​()\n" + + "deltaProperty()\n" + " \n" + "\n" + "\n" + "java.util.List<java.lang.String>\n" + "" - + "gammaProperty​()\n" + + "gammaProperty()\n" + " " ); } diff -r ce5fd3ba3fea -r 355349babaf4 test/langtools/jdk/javadoc/doclet/testLambdaFeature/TestLambdaFeature.java --- a/test/langtools/jdk/javadoc/doclet/testLambdaFeature/TestLambdaFeature.java Wed Sep 27 14:23:41 2017 -0700 +++ b/test/langtools/jdk/javadoc/doclet/testLambdaFeature/TestLambdaFeature.java Wed Sep 27 16:47:07 2017 -0700 @@ -23,7 +23,7 @@ /* * @test - * @bug 8004893 8022738 8029143 8175200 + * @bug 8004893 8022738 8029143 8175200 8186332 * @summary Make sure that the lambda feature changes work fine in * javadoc. * @author bpatel @@ -55,7 +55,7 @@ checkOutput("pkg/A.html", true, "default void", - "
default void defaultMethod​()
", + "
default void defaultMethod()
", "" + "All Methods " + "" @@ -83,7 +83,7 @@ checkOutput("pkg/A.html", false, "default default void", - "
default default void defaultMethod​()
"); + "
default default void defaultMethod()
"); checkOutput("pkg/B.html", false, "default void", diff -r ce5fd3ba3fea -r 355349babaf4 test/langtools/jdk/javadoc/doclet/testLiteralCodeInPre/TestLiteralCodeInPre.java --- a/test/langtools/jdk/javadoc/doclet/testLiteralCodeInPre/TestLiteralCodeInPre.java Wed Sep 27 14:23:41 2017 -0700 +++ b/test/langtools/jdk/javadoc/doclet/testLiteralCodeInPre/TestLiteralCodeInPre.java Wed Sep 27 16:47:07 2017 -0700 @@ -23,7 +23,7 @@ /* * @test - * @bug 8002387 8014636 8078320 8175200 + * @bug 8002387 8014636 8078320 8175200 8186332 * @summary Improve rendered HTML formatting for {@code} * @library ../lib * @modules jdk.javadoc/jdk.javadoc.internal.tool @@ -47,19 +47,19 @@ checkExit(Exit.OK); checkOutput("pkg/Test.html", true, - "no_pre​()\n" + "no_pre()\n" + "
abcdefghi
", - "no_pre_extra_whitespace​()\n" + "no_pre_extra_whitespace()\n" + "
abc def ghi
", - "in_pre​()\n" + "in_pre()\n" + "
 abc def  ghi
", - "pre_after_text​()\n" + "pre_after_text()\n" + "
xyz
 abc def  ghi
", - "after_pre​()\n" + "after_pre()\n" + "
xyz
 pqr 
abc def ghi
", - "back_in_pre​()\n" + "back_in_pre()\n" + "
xyz
 pqr 
mno
 abc def  ghi
", - "typical_usage_code​()\n" + "typical_usage_code()\n" + "
Lorem ipsum dolor sit amet, consectetur adipiscing elit.\n" + " Example:
\n"
                 + "   line 0 @Override\n"
@@ -68,7 +68,7 @@
                 + "   line 3 }\n"
                 + " 
\n" + " and so it goes.
", - "typical_usage_literal​()\n" + "typical_usage_literal()\n" + "
Lorem ipsum dolor sit amet, consectetur adipiscing elit.\n" + " Example:
\n"
                 + "   line 0 @Override\n"
@@ -77,7 +77,7 @@
                 + "   line 3 }\n"
                 + " 
\n" + " and so it goes.
", - "recommended_usage_literal​()\n" + "recommended_usage_literal()\n" + "
Lorem ipsum dolor sit amet, consectetur adipiscing elit.\n" + " Example:
\n"
                 + "   line 0 @Override\n"
@@ -89,7 +89,7 @@
                 + " 
\n"
                 + " id           \n"
                 + " 
", - "
public void htmlAttrInPre1​()
\n" + "
public void htmlAttrInPre1()
\n" + "
More html tag outliers.\n" + "
\n"
                 + " @Override\n"
diff -r ce5fd3ba3fea -r 355349babaf4 test/langtools/jdk/javadoc/doclet/testMemberSummary/TestMemberSummary.java
--- a/test/langtools/jdk/javadoc/doclet/testMemberSummary/TestMemberSummary.java	Wed Sep 27 14:23:41 2017 -0700
+++ b/test/langtools/jdk/javadoc/doclet/testMemberSummary/TestMemberSummary.java	Wed Sep 27 16:47:07 2017 -0700
@@ -23,7 +23,7 @@
 
 /*
  * @test
- * @bug      4951228 6290760 8025633 8026567 8081854 8162363 8175200 8177417
+ * @bug      4951228 6290760 8025633 8026567 8081854 8162363 8175200 8177417 8186332
  * @summary  Test the case where the overriden method returns a different
  *           type than the method in the child class.  Make sure the
  *           documentation is inherited but the return type isn't.
@@ -52,12 +52,12 @@
                 // Check return type in member summary.
                 "PublicChild\n"
                 + ""
-                + "returnTypeTest​()",
+                + "returnTypeTest()",
                 // Check return type in member detail.
                 "
public "
-                + "PublicChild returnTypeTest​()
", + + "PublicChild returnTypeTest()
", "" - + "PublicChild​()"); + + "PublicChild()"); checkOutput("pkg/PrivateParent.html", true, "private \n" diff -r ce5fd3ba3fea -r 355349babaf4 test/langtools/jdk/javadoc/doclet/testNewLanguageFeatures/TestNewLanguageFeatures.java --- a/test/langtools/jdk/javadoc/doclet/testNewLanguageFeatures/TestNewLanguageFeatures.java Wed Sep 27 14:23:41 2017 -0700 +++ b/test/langtools/jdk/javadoc/doclet/testNewLanguageFeatures/TestNewLanguageFeatures.java Wed Sep 27 16:47:07 2017 -0700 @@ -24,7 +24,7 @@ /* * @test * @bug 4789689 4905985 4927164 4827184 4993906 5004549 7025314 7010344 8025633 8026567 8162363 - * 8175200 + * 8175200 8186332 * @summary Run Javadoc on a set of source files that demonstrate new * language features. Check the output to ensure that the new * language features are properly documented. @@ -549,7 +549,7 @@ + "=\"Constructor Annotation\",\n" + " " + "required=1994)\n" - + "public AnnotationTypeUsage​()", + + "public AnnotationTypeUsage()", // METHOD "
@AnnotationType("
@@ -557,7 +557,7 @@
                 + "=\"Method Annotation\",\n"
                 + "                "
                 + "required=1994)\n"
-                + "public void method​()
", + + "public void method()", // METHOD PARAMS "
public void methodWithParams​("
                 + ""
@@ -630,11 +630,11 @@
                 // CONSTRUCTOR
                 "@AnnotationTypeUndocumented(optional=\"Constructor Annotation\",\n"
                 + "                required=1994)\n"
-                + "public AnnotationTypeUsage​()",
+                + "public AnnotationTypeUsage()",
                 // METHOD
                 "@AnnotationTypeUndocumented(optional=\"Method Annotation\",\n"
                 + "                required=1994)\n"
-                + "public void method​()");
+                + "public void method()");
 
         //=================================
         // Make sure annotation types do not
diff -r ce5fd3ba3fea -r 355349babaf4 test/langtools/jdk/javadoc/doclet/testOptions/TestOptions.java
--- a/test/langtools/jdk/javadoc/doclet/testOptions/TestOptions.java	Wed Sep 27 14:23:41 2017 -0700
+++ b/test/langtools/jdk/javadoc/doclet/testOptions/TestOptions.java	Wed Sep 27 16:47:07 2017 -0700
@@ -23,7 +23,7 @@
 
 /*
  * @test
- * @bug      4749567 8071982 8175200
+ * @bug      4749567 8071982 8175200 8186332
  * @summary  Test the output for -header, -footer, -nooverview, -nodeprecatedlist, -nonavbar, -notree, -stylesheetfile options.
  * @author   Bhavesh Patel
  * @library  ../lib
@@ -148,7 +148,7 @@
                 "
public java.lang.Object someProperty
", "
public java.lang.Object someProperty​()
"); + + "\"../src-html/linksource/Properties.html#line.31\">someProperty()
"); checkOutput("src-html/linksource/Properties.html", true, "Source code", @@ -161,9 +161,9 @@ "
public int "
                 + "field
", "
public "
-                + "SomeClass​()
", + + "SomeClass()", "
public int "
-                + "method​()
"); + + "method()"); checkOutput("src-html/linksource/SomeClass.html", true, "Source code", diff -r ce5fd3ba3fea -r 355349babaf4 test/langtools/jdk/javadoc/doclet/testOverridenMethods/TestBadOverride.java --- a/test/langtools/jdk/javadoc/doclet/testOverridenMethods/TestBadOverride.java Wed Sep 27 14:23:41 2017 -0700 +++ b/test/langtools/jdk/javadoc/doclet/testOverridenMethods/TestBadOverride.java Wed Sep 27 16:47:07 2017 -0700 @@ -23,7 +23,7 @@ /* * @test - * @bug 8174839 8175200 + * @bug 8174839 8175200 8186332 * @summary Bad overriding method should not crash * @library ../lib * @modules jdk.javadoc/jdk.javadoc.internal.tool @@ -52,7 +52,7 @@ checkOutput("pkg4/Foo.html", true, "
  • \n" + "

    toString

    \n" - + "
    public void toString​()
    \n" + + "
    public void toString()
    \n" + "
    Why can't I do this ?
    \n" + "
  • "); } diff -r ce5fd3ba3fea -r 355349babaf4 test/langtools/jdk/javadoc/doclet/testSummaryTag/TestSummaryTag.java --- a/test/langtools/jdk/javadoc/doclet/testSummaryTag/TestSummaryTag.java Wed Sep 27 14:23:41 2017 -0700 +++ b/test/langtools/jdk/javadoc/doclet/testSummaryTag/TestSummaryTag.java Wed Sep 27 16:47:07 2017 -0700 @@ -25,7 +25,7 @@ /* * @test - * @bug 8173425 + * @bug 8173425 8186332 * @summary tests for the summary tag behavior * @library ../lib * @modules jdk.javadoc/jdk.javadoc.internal.tool @@ -87,7 +87,7 @@ checkOutput("p1/A.html", true, "
  • \n" + "

    m3

    \n" - + "
    public void m3​()
    \n" + + "
    public void m3()
    \n" + "
    First sentence some text maybe second sentence.
    \n" + "
  • \n" ); diff -r ce5fd3ba3fea -r 355349babaf4 test/langtools/jdk/javadoc/doclet/testTypeAnnotations/TestTypeAnnotations.java --- a/test/langtools/jdk/javadoc/doclet/testTypeAnnotations/TestTypeAnnotations.java Wed Sep 27 14:23:41 2017 -0700 +++ b/test/langtools/jdk/javadoc/doclet/testTypeAnnotations/TestTypeAnnotations.java Wed Sep 27 16:47:07 2017 -0700 @@ -23,7 +23,7 @@ /* * @test - * @bug 8005091 8009686 8025633 8026567 6469562 8071982 8071984 8162363 8175200 + * @bug 8005091 8009686 8025633 8026567 6469562 8071982 8071984 8162363 8175200 8186332 * @summary Make sure that type annotations are displayed correctly * @author Bhavesh Patel * @library ../lib @@ -155,17 +155,17 @@ checkOutput("typeannos/MtdDefaultScope.html", true, "
    public <T> @MRtnA java.lang.String"
    -                + " method​()
    ", + + " method()", // When JDK-8068737 is fixed, we should change the order "
    "
                     + "@MRtnA java.lang.String "
                     + "@MRtnB [] "
                     + "@MRtnA []"
    -                + " array2Deep​()
    ", + + " array2Deep()", "
    @MRtnA java.lang.String[][] array2​()
    "); + + "typeannos\">@MRtnA java.lang.String[][] array2()"); checkOutput("typeannos/MtdModifiedScoped.html", true, "
    public final @MRtnB java.lang.String>,@MRtnB java."
    -                + "lang.String> nestedMtdParameterized​()
    "); + + "lang.String> nestedMtdParameterized()"); // Test for type annotations on method type parameters (MethodTypeParameters.java). checkOutput("typeannos/UnscopedUnmodified.html", true, "
    <K extends @MTyParamA java.lang.String>"
    -                + " void methodExtends​()
    ", + + " void methodExtends()", "
    <K extends @MTyParamA "
                     + "MtdTyParameterized<@MTyParamB java.lang.String"
    -                + ">> void nestedExtends​()
    "); + + ">> void nestedExtends()"); checkOutput("typeannos/PublicModifiedMethods.html", true, "
    public final <K extends @MTyParamA "
    -                + "java.lang.String> void methodExtends​()
    ", + + "java.lang.String> void methodExtends()", "
    public final <K extends @MTyParamA "
    @@ -204,7 +204,7 @@
                     + "typeannos/MtdTyParameterized.html\" title=\"class in typeannos\">"
                     + "MtdTyParameterized<@MTyParamB java.lang.String"
    -                + ">> void dual​()
    "); + + ">> void dual()"); // Test for type annotations on parameters (Parameters.java). checkOutput("typeannos/Parameters.html", true, @@ -236,11 +236,11 @@ // Test for type annotations on throws (Throws.java). checkOutput("typeannos/ThrDefaultUnmodified.html", true, - "
    void oneException​()\n"
    +                "
    void oneException()\n"
                     + "           throws @ThrA java.lang.Exception
    ", - "
    void twoExceptions​()\n"
    +                "
    void twoExceptions()\n"
                     + "            throws @ThrA java.lang.RuntimeException,\n"
                     + "                   @ThrA java.lang.Exception
    "); checkOutput("typeannos/ThrWithValue.html", true, - "
    void oneException​()\n"
    +                "
    void oneException()\n"
                     + "           throws @ThrB("
                     + "\"m\") java.lang.Exception
    ", - "
    void twoExceptions​()\n"
    +                "
    void twoExceptions()\n"
                     + "            throws @ThrB("
                     + "\"m\") java.lang.RuntimeException,\n"
    @@ -275,7 +275,7 @@
                     "
    <K,"
                     + "@TyParaA V extends @TyParaA "
    -                + "java.lang.String> void secondAnnotated​()
    " + + "java.lang.String> void secondAnnotated()
    " ); // Test for type annotations on wildcard type (Wildcards.java). @@ -288,7 +288,7 @@ "
    MyList<? super @WldA java.lang.String>"
    -                + " returnWcSuper​()
    "); + + " returnWcSuper()
    "); checkOutput("typeannos/BoundWithValue.html", true, "
    void wcSuper​(MyList<? extends @WldB("
                     + "\"m\") java.lang.String"
    -                + "> returnWcExtends​()
    "); + + "> returnWcExtends()
    "); // Test for receiver annotations (Receivers.java). checkOutput("typeannos/DefaultUnmodified.html", true, @@ -396,7 +396,7 @@ + "\"../typeannos/RepConstructorB.html\" title=\"annotation in typeannos" + "\">@RepConstructorB @RepConstructorB\n" - + "RepeatingOnConstructor​()
    ", + + "RepeatingOnConstructor()
    ", "
    @RepConstructorA (package private) java.lang.String\n"
                     + "test1​()",
    +                + "\"../typeannos/RepeatingOnMethod.html#test1--\">test1()",
     
                     "(package private) @RepTypeUseA @RepTypeUseB java.lang.String"
                     + "\n"
                     + "test2"
    -                + "​()",
    +                + "()",
     
                     "(package private) @RepTypeUseA @RepTypeUseB java.lang.String"
                     + "\n"
                     + "test3"
    -                + "​()",
    +                + "()",
     
                     "(package private) @RepAllContextsA "
                     + "@RepAllContextsB java.lang.String\n"
                     + "test4​()",
    +                + "#test4--\">test4()",
     
                     "test5"
    @@ -621,13 +621,13 @@
                     + "\"annotation in typeannos\">@RepMethodA\n@RepMethodB "
    -                + "@RepMethodB\njava.lang.String test1​()",
    +                + "@RepMethodB\njava.lang.String test1()",
     
                     ""
                     + "@RepTypeUseA @RepTypeUseA @RepTypeUseB @RepTypeUseB java.lang.String test2​()",
    +                + "title=\"annotation in typeannos\">@RepTypeUseB java.lang.String test2()",
     
                     ""
                     + "@RepMethodA @RepTypeUseA "
                     + "@RepTypeUseB @RepTypeUseB java.lang.String test3​()",
    +                + "\"annotation in typeannos\">@RepTypeUseB java.lang.String test3()",
     
                     ""
                     + "@RepAllContextsA @RepAllContextsA "
                     + "@RepAllContextsB @RepAllContextsB java.lang.String test4​()",
    +                + "title=\"annotation in typeannos\">@RepAllContextsB java.lang.String test4()",
     
                     "java.lang.String test5​(@RepTypeUseA (package private) java.lang.String\n"
                     + "test​()",
    +                + "test--\">test()",
     
                     "java.lang.String test​(@RepTypeUseA @RepMethodA\n@RepMethodB "
    -                + "@RepMethodB\nvoid test​()");
    +                + "@RepMethodB\nvoid test()");
         }
     }
    diff -r ce5fd3ba3fea -r 355349babaf4 test/langtools/jdk/javadoc/doclet/testUseOption/TestUseOption.java
    --- a/test/langtools/jdk/javadoc/doclet/testUseOption/TestUseOption.java	Wed Sep 27 14:23:41 2017 -0700
    +++ b/test/langtools/jdk/javadoc/doclet/testUseOption/TestUseOption.java	Wed Sep 27 16:47:07 2017 -0700
    @@ -23,7 +23,7 @@
     
     /*
      * @test
    - * @bug 4496290 4985072 7006178 7068595 8016328 8050031 8048351 8081854 8071982 8162363 8175200
    + * @bug 4496290 4985072 7006178 7068595 8016328 8050031 8048351 8081854 8071982 8162363 8175200 8186332
      * @summary A simple test to ensure class-use files are correct.
      * @author jamieh
      * @library ../lib
    @@ -134,7 +134,7 @@
                 "void\nC1."
                 + "methodInC1ThrowsThrowable"
    -            + "​()"
    +            + "()"
             );
         }