mbeckerle commented on a change in pull request #456:
URL: https://github.com/apache/incubator-daffodil/pull/456#discussion_r522403119



##########
File path: 
daffodil-udf/src/main/java/org/apache/daffodil/udf/UserDefinedFunctionIdentification.java
##########
@@ -28,11 +28,21 @@
  *
  * It must have the name and namespaceURI fields initialized with the namespace
  * and name callers would be expected to use in the schema.
- *
  */
 @Retention(RetentionPolicy.RUNTIME)
 @Target(ElementType.TYPE)
 public @interface UserDefinedFunctionIdentification {
+  /**
+   * Get the name of the user defined function
+   *
+   * @return the name of the user defined function
+   */
   String name();
+
+  /**
+   * Get the namespace URI of the user defined function

Review comment:
       Ok then. Will one-liners work. 
   ```
   /** @returns the value */
   ```
   Might work. 1 line vs. 5. 




----------------------------------------------------------------
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.

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


Reply via email to