Added support for synchronized keyword in some Declaration subclasses.

Author: cpythoud

src/org/jcodegen/java/ConstructorDeclaration.java

@@ -34,6 +34,11 @@ public ConstructorDeclaration markAsAbstract() {
         throw new UnsupportedOperationException("Constructors cannot be abstract.");
     }
 
+    @Override
+    public ConstructorDeclaration markAsSynchronized() {
+        throw new UnsupportedOperationException("Constructors cannot be synchronized.");
+    }
+
 
     @Override
     public String toString() {

src/org/jcodegen/java/Declaration.java

@@ -5,6 +5,11 @@
  */
 public abstract class Declaration<T extends Declaration<T>> extends JavaCodeBlock<T> {
 
+    // TODO :
+    // revisit subclass hierarchy to prevent creation of illegal constructs in java,
+    // like synchronized constructors or abstract fields,
+    // at compile time instead of throwing UnsupportedOperationExceptions
+
     private final String name;
     //private final String codeEntity;
 
@@ -13,6 +18,7 @@ public abstract class Declaration<T extends Declaration<T>> extends JavaCodeBloc
     private boolean isAbstract = false;
     private boolean isFinal = false;
     private boolean isStatic = false;
+    private boolean isSynchronized = false;
 
     private String annotations = null;
 
@@ -49,12 +55,20 @@ public T markAsFinal() {
 
     public T markAsStatic() {
         if (isAbstract)
-            throw new IllegalArgumentException(getAbstractAndFinalErrorMessage());
+            throw new IllegalArgumentException(getAbstractAndStaticErrorMessage());
 
         isStatic = true;
         return getThis();
     }
 
+    public T markAsSynchronized() {
+        if (isAbstract)
+            throw new IllegalArgumentException(getAbstractAndSynchronizedErrorMessage());
+
+        isSynchronized = true;
+        return getThis();
+    }
+
     public T annotate(final String annotations) {
         this.annotations = annotations;
         return getThis();
@@ -73,6 +87,10 @@ protected String getAbstractAndStaticErrorMessage() {
         return getKeyword() + " cannot be abstract AND static";
     }
 
+    protected String getAbstractAndSynchronizedErrorMessage() {
+        return getKeyword() + " cannot be abstract AND synchronized";
+    }
+
     protected void appendAnnotations(final StringBuilder buf) {
         if (annotations != null) {
             buf.append(getTabs());
@@ -93,5 +111,7 @@ protected void appendDeclarationStart(final StringBuilder buf) {
             buf.append("static ");
         if (isFinal)
             buf.append("final ");
+        if (isSynchronized)
+            buf.append("synchronized ");
     }
 }

src/org/jcodegen/java/JavaClass.java

@@ -42,6 +42,12 @@ public JavaClass implementsGenericInterface(final String implementedInterface, f
     }
 
 
+    @Override
+    public JavaClass markAsSynchronized() {
+        throw new UnsupportedOperationException("Class cannot be synchronized.");
+    }
+
+
     @Override
     public String toString() {
         final StringBuilder buf = new StringBuilder();

src/org/jcodegen/java/VarDeclaration.java

@@ -52,6 +52,16 @@ protected VarDeclaration getThis() {
         return this;
     }
 
+    @Override
+    public VarDeclaration markAsAbstract() {
+        throw new UnsupportedOperationException("Field or variable cannot be abstract.");
+    }
+
+    @Override
+    public VarDeclaration markAsSynchronized() {
+        throw new UnsupportedOperationException("Field or variable cannot be synchronized.");
+    }
+
 
     @Override
     public String toString() {