PUBLIC COMMENTS
I have fixed this bug. When a bad parameter name is passed to @param, the following warning is printed:
@param argument "paramName" is not a parameter name.
When @param is used more than once for the same parameter, the following warning is printed:
Parameter "paramName" is documented more than once.
###@###.### 2002-07-18
|
EVALUATION
What is happening here is that starting with 1.4.0, the @param tag requires
its first argument to be the parameter name. In this case, the user has
specified the parameter type instead.
Prior to 1.4.0, the test method parameters were documented as:
Parameters:
name - -String
age - -int
With 1.4.1, however, the Parameters heading and entries are missing altogether,
with no warning. Javadoc should print a warning when the parameter
name specified by @param does not appear in the declaration's
parameter list.
Suggest a warning such as (of course, the filename and line number
should be included):
warning: @param argument "String" is not a parameter name
Committed to mantis.
Changed synopsis from:
javadoc does not generate @param
to this:
Does not generate @param when its first arg is not parameter name
###@###.### 2002-05-30
Changed synopsis from:
Does not generate @param when its first arg is not parameter name
to this:
Does not warn when first argument of @param is not parameter name
Removed "REGRESSION" from synopsis, as not generating @param when there
is a mismatch is a new feature.
###@###.### 2002-05-31
This bug fix should also handle the case where there is an extra @param
tag listed, as the first arg to @param would not correspond to a parameter name.
This was discussed on the Javadoc Forum subject "@param". For example:
public class ParamBug {
/**
* Space pad a String.
@param S String to pad.
@param length Desired minimum length.
@param end Should the padding be at the end.
@param extra the extra param
*/
public static String pad(String S, int length, boolean end) {
}
}
###@###.### 2002-05-31
I have fixed this bug. The bug fix is pending review.
###@###.### 2002-07-08
|