Fix #357

pmd-java/src/main/resources/rulesets/java/design.xml

@@ -1425,8 +1425,10 @@ and unintentional empty constructors.
           <property name="xpath">
               <value>
     <![CDATA[
-//ConstructorDeclaration[@Private='false'][count(BlockStatement) = 0 and ($ignoreExplicitConstructorInvocation = 'true' or not(ExplicitConstructorInvocation)) and @containsComment = 'false']
- ]]>
+//ConstructorDeclaration[@Private='false']
+                        [count(BlockStatement) = 0 and ($ignoreExplicitConstructorInvocation = 'true' or not(ExplicitConstructorInvocation)) and @containsComment = 'false']
+                        [not(../Annotation/MarkerAnnotation/Name[@Image='Inject'])]
+]]>
              </value>
           </property>
           <property name="ignoreExplicitConstructorInvocation" type="Boolean" description="Ignore explicit constructor invocation when deciding whether constructor is empty or not" value="false"/>

pmd-java/src/test/resources/net/sourceforge/pmd/lang/java/rule/design/xml/UncommentedEmptyConstructor.xml

@@ -170,4 +170,19 @@ public class WebAgent {
 }
         ]]></code>
     </test-code>
+
+    <test-code>
+        <description>#357 UncommentedEmptyConstructor consider annotations on Constructor</description>
+        <expected-problems>0</expected-problems>
+        <code><![CDATA[
+        import javax.inject.Inject;
+
+        public class MyClass {
+          @Inject MyClass() {
+
+          }
+        }
+        ]]>
+        </code>
+    </test-code>
 </test-data>