Correct javadocs.
git-svn-id: https://svn.apache.org/repos/asf/felix/trunk@601722 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/scrplugin/src/main/java/org/apache/felix/scrplugin/SCRDescriptorMojo.java b/scrplugin/src/main/java/org/apache/felix/scrplugin/SCRDescriptorMojo.java
index 3479605..9e29d8f 100644
--- a/scrplugin/src/main/java/org/apache/felix/scrplugin/SCRDescriptorMojo.java
+++ b/scrplugin/src/main/java/org/apache/felix/scrplugin/SCRDescriptorMojo.java
@@ -404,7 +404,6 @@
* @param services
* @param component
* @param description
- * @return
* @throws MojoExecutionException
*/
protected void doServices(JavaTag[] services, Component component, JavaClassDescription description)
diff --git a/scrplugin/src/main/java/org/apache/felix/scrplugin/om/AbstractObject.java b/scrplugin/src/main/java/org/apache/felix/scrplugin/om/AbstractObject.java
index 3d7c23b..92d5a94 100644
--- a/scrplugin/src/main/java/org/apache/felix/scrplugin/om/AbstractObject.java
+++ b/scrplugin/src/main/java/org/apache/felix/scrplugin/om/AbstractObject.java
@@ -35,8 +35,7 @@
protected String getMessage(String message) {
if ( this.tag == null ) {
return message;
- } else {
- return "@" + this.tag.getName() + ": " + message + " (" + this.tag.getSourceLocation() + ")";
}
+ return "@" + this.tag.getName() + ": " + message + " (" + this.tag.getSourceLocation() + ")";
}
}
diff --git a/scrplugin/src/main/java/org/apache/felix/scrplugin/om/Component.java b/scrplugin/src/main/java/org/apache/felix/scrplugin/om/Component.java
index d6a6b72..aab61c8 100644
--- a/scrplugin/src/main/java/org/apache/felix/scrplugin/om/Component.java
+++ b/scrplugin/src/main/java/org/apache/felix/scrplugin/om/Component.java
@@ -90,7 +90,7 @@
}
/**
- * @return
+ * @return All properties of this component.
*/
public List getProperties() {
return this.properties;
diff --git a/scrplugin/src/main/java/org/apache/felix/scrplugin/om/metatype/AttributeDefinition.java b/scrplugin/src/main/java/org/apache/felix/scrplugin/om/metatype/AttributeDefinition.java
index 2a96ba0..8f63f66 100644
--- a/scrplugin/src/main/java/org/apache/felix/scrplugin/om/metatype/AttributeDefinition.java
+++ b/scrplugin/src/main/java/org/apache/felix/scrplugin/om/metatype/AttributeDefinition.java
@@ -23,7 +23,7 @@
public class AttributeDefinition {
public static final String DEFAULT_TYPE = "String";
-
+
protected String id;
protected String type = DEFAULT_TYPE;
@@ -74,7 +74,7 @@
if (values != null) {
this.defaultValue = null;
this.defaultMultiValue = values;
- if (values != null && values.length > 0 && this.cardinality == null ) {
+ if (values.length > 0 && this.cardinality == null ) {
this.cardinality = new Integer(Integer.MAX_VALUE);
}
}
diff --git a/scrplugin/src/main/java/org/apache/felix/scrplugin/tags/JavaClassDescription.java b/scrplugin/src/main/java/org/apache/felix/scrplugin/tags/JavaClassDescription.java
index e3b00ce..cf15c48 100644
--- a/scrplugin/src/main/java/org/apache/felix/scrplugin/tags/JavaClassDescription.java
+++ b/scrplugin/src/main/java/org/apache/felix/scrplugin/tags/JavaClassDescription.java
@@ -46,14 +46,15 @@
/**
* Get the description for the parent class.
- * @return
+ * @return The description or <code>null</code> if this class is the
+ * Object class.
* @throws MojoExecutionException
*/
JavaClassDescription getSuperClass() throws MojoExecutionException;
/**
* Get the name of the described class.
- * @return
+ * @return The name of the described class.
*/
String getName();
@@ -65,7 +66,7 @@
* Search for a method with the given signature.
* @param name
* @param parameters
- * @return
+ * @return A descriptor for the method or <code>null</code>
* @throws MojoExecutionException
*/
JavaMethod getMethodBySignature(String name, String[] parameters)
@@ -73,19 +74,19 @@
/**
* Is this class public?
- * @return
+ * @return True if this class is public.
*/
boolean isPublic();
/**
* Is this class abstract?
- * @return
+ * @return True if this class is abstract.
*/
boolean isAbstract();
/**
* Is this class an interface?
- * @return
+ * @return True if this is an interface.
*/
boolean isInterface();
@@ -94,7 +95,7 @@
/**
* Is this class of the type?
* @param type
- * @return
+ * @return True if this class is of the type.
* @throws MojoExecutionException
*/
boolean isA(String type) throws MojoExecutionException;
diff --git a/scrplugin/src/main/java/org/apache/felix/scrplugin/tags/JavaClassDescriptorManager.java b/scrplugin/src/main/java/org/apache/felix/scrplugin/tags/JavaClassDescriptorManager.java
index 4be1e6d..20f1cdc 100644
--- a/scrplugin/src/main/java/org/apache/felix/scrplugin/tags/JavaClassDescriptorManager.java
+++ b/scrplugin/src/main/java/org/apache/felix/scrplugin/tags/JavaClassDescriptorManager.java
@@ -271,7 +271,7 @@
/**
* Return all source descriptions of this project.
- * @return
+ * @return All contained java class descriptions.
*/
public JavaClassDescription[] getSourceDescriptions() {
final JavaClassDescription[] descs = new JavaClassDescription[this.sources.length];
@@ -284,7 +284,7 @@
/**
* Get a java class description for the class.
* @param className
- * @return
+ * @return The java class description.
* @throws MojoExecutionException
*/
public JavaClassDescription getJavaClassDescription(String className)
@@ -309,9 +309,7 @@
throw new MojoExecutionException("Unable to load class " + className);
}
}
- if ( result != null ) {
- this.javaClassDescriptions.put(className, result);
- }
+ this.javaClassDescriptions.put(className, result);
}
return result;
}