tuxji commented on code in PR #875: URL: https://github.com/apache/daffodil/pull/875#discussion_r1024198173
########## daffodil-cli/src/main/scala/org/apache/daffodil/Main.scala: ########## @@ -117,6 +116,7 @@ import org.apache.daffodil.xml.QName import org.apache.daffodil.xml.RefQName import org.apache.daffodil.xml.XMLUtils import org.apache.daffodil.xml.DFDLCatalogResolver +// import org.apache.daffodil.processors.HasSetDebugger Review Comment: I looked at TDMLRunner.scala and I don't think it has any good reason why DFDLTestSuite needs to extend HasSetDebugger. If you remove the import, the extends HasSetDebugger, and the override keyword from the `setDebugger` and `setDebugging` methods and you still can build the code and run the tests without errors, we'll know for sure. -- 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]
