mbien commented on code in PR #5561:
URL: https://github.com/apache/netbeans/pull/5561#discussion_r1117381110


##########
harness/nbjunit/src/org/netbeans/junit/ParametricTestSuite.java:
##########
@@ -36,23 +36,28 @@ public ParametricTestSuite() {
     
     /**
      * Constructs a ParametricTestSuite with the given name.
+     * @param name name of the suite
      */
     public ParametricTestSuite(String name){
         super(name);
     }
     
     /**
      * Returns an array of testcases for the given parameter.
+     * @param parameter parametre to retrieve the test case frome
+     * @return array of testcases
      */
     protected abstract ParametricTestCase[] cases(Object parameter);
     /**
      * Returns an array of parameters for this suite.
+     * @return array or parameter
      */
     protected abstract Object[] getParameters();
     
     /**
      * Factory method returns a new instance of a testcases.
      * Overrides the basic method so that it's needless any more.
+     * @return test case

Review Comment:
   i gotta say I hate this kind of doc :)
   
   This isn't adding any information it is just auto generated noise. Is there 
no way to turn off the warnings for things like this? This doesn't actually 
make the doc any better.



##########
ide/libs.git/src/org/netbeans/libs/git/GitClient.java:
##########
@@ -387,10 +391,12 @@ public boolean catIndexEntry (File file, int stage, 
java.io.OutputStream out, Pr
     }
 
     /**
-     * Checks out the index into the working copy root. Does not move current 
HEAD.
+     * Checks out the index into the working copy root.Does not move current 
HEAD.

Review Comment:
   intended?



##########
java/classfile/src/org/netbeans/modules/classfile/Field.java:
##########
@@ -147,8 +148,9 @@ void setTypeSignature(String sig) {
     }
 
     /**
-     * Returns al<ClassName,Annotation>l runtime annotations defined for this 
field.  Inherited
+     * Returns all &lt;ClassName,Annotation&gt; runtime annotations defined 
for this field.  Inherited
      * annotations are not included in this collection.
+     * @return all annotation

Review Comment:
   +s



##########
harness/nbjunit/src/org/netbeans/junit/ParametricTestSuite.java:
##########
@@ -36,23 +36,28 @@ public ParametricTestSuite() {
     
     /**
      * Constructs a ParametricTestSuite with the given name.
+     * @param name name of the suite
      */
     public ParametricTestSuite(String name){
         super(name);
     }
     
     /**
      * Returns an array of testcases for the given parameter.
+     * @param parameter parametre to retrieve the test case frome
+     * @return array of testcases
      */
     protected abstract ParametricTestCase[] cases(Object parameter);
     /**
      * Returns an array of parameters for this suite.
+     * @return array or parameter

Review Comment:
   array **of** parameters



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists

Reply via email to