Re: [PR] Remove uses of ReflectiveCalls from Code [daffodil]

["mbeckerle (via GitHub)" <gi...@apache.org>]

mbeckerle commented on code in PR #1452:
URL: https://github.com/apache/daffodil/pull/1452#discussion_r1986111257


##########
daffodil-io/src/main/scala/org/apache/daffodil/io/processors/charset/BitsCharsetDefinition.scala:
##########
@@ -16,13 +16,18 @@
  */
 package org.apache.daffodil.io.processors.charset
 
+import org.apache.daffodil.lib.util.hasNameDefined
+
 /**
  * These are the classes which must be dynamically loaded in order to add a charset implementation
  * to Daffodil. All charsets must implement this class and be added to the 
  * org.apache.daffodil.runtime1.processors.charset.BitsCharsetDefinition file in 
  * daffodil-io/src/main/resources/META-INF/services. name() must return a fully capitalized string
  */
-abstract class BitsCharsetDefinition(charset: BitsCharset, alias: Option[String] = None) {
+abstract class BitsCharsetDefinition(
+  charset: BitsCharset,
+  alias: Option[String] = None
+) extends hasNameDefined {

Review Comment:
   Should be "H" initial caps on  trait name. 
   
   I suggest remove the suffix "Defined" and just call it "HasName" or "HasNameMethod"



-- 
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: commits-unsubscribe@daffodil.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org