stevedlawrence commented on a change in pull request #532:
URL: https://github.com/apache/daffodil/pull/532#discussion_r617604757



##########
File path: daffodil-cli/src/main/scala/org/apache/daffodil/Main.scala
##########
@@ -502,11 +502,47 @@ class CLIConf(arguments: Array[String]) extends 
scallop.ScallopConf(arguments)
     val info = tally(descr = "increment test result information output level, 
one level for each -i")
   }
 
+  // Generate Subcommand Options
+  object generate extends scallop.Subcommand("generate") {
+    descr("generate <language> code from a DFDL schema")
+
+    banner("""|Usage: daffodil [GLOBAL_OPTS] generate <language> 
[SUBCOMMAND_OPTS]
+              |""".stripMargin)
+    shortSubcommandsHelp()
+    footer("""|
+              |Run 'daffodil generate <language> --help' for subcommand 
specific options""".stripMargin)
+
+    object c extends scallop.Subcommand("c") {

Review comment:
       "c-runtime2" makes me think there's a c-runtime1. Maybe just plain "c" 
is reasonable, considering 
[DAFFODIL-2406](https://issues.apache.org/jira/browse/DAFFODIL-2406) suggests 
renaming things to "backend-c-generator". The "generator" part is assumed in 
the "daffodil generator" subcommand, and backend is also implied. So maybe just 
plain "c" is the best option?




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