RFR 8012681: Commit for JDK-8012656 breaks tl build Was: can't build current jdk8/tl tip

[Joel Borggrén-Franck]

Hi

On 04/19/2013 09:30 AM, Alan Bateman wrote:> On 19/04/2013 08:08, Peter 
Levart wrote:
 >> Hi,
 >>
 >> I get the following compile-time error:
 >>
 >> 
/home/peter/work/hg/jdk8-tl/langtools/src/share/classes/com/sun/tools/javadoc/FieldDocImpl.java:264: 

 >> error: illegal start of expression
 >>         if (qualifiedName == null) }
 > I'm seeing the same thing, looks like the typo crept in with this change
 > last night:
 >
 > http://hg.openjdk.java.net/jdk8/tl/langtools/rev/95d29b99e5b3
 >

See inline patch to fix this. Since TL isn't building I think we can review 
this on core-libs-dev although technically a compiler issue.

With this fix langtools builds and all langtools tests passes.

cheers
/Joel

diff --git a/src/share/classes/com/sun/tools/javadoc/FieldDocImpl.java 
b/src/share/classes/com/sun/tools/javadoc/FieldDocImpl.java
--- a/src/share/classes/com/sun/tools/javadoc/FieldDocImpl.java
+++ b/src/share/classes/com/sun/tools/javadoc/FieldDocImpl.java
@@ -261,7 +261,7 @@
      private String name;

      public String qualifiedName() {
-        if (qualifiedName == null) }
+        if (qualifiedName == null) {
              qualifiedName = sym.enclClass().getQualifiedName() + "." + name();
          }
          return qualifiedName;