mbeckerle commented on a change in pull request #12: Revised daffodil-io module
to require passing in a FormatInfo object.
URL: https://github.com/apache/incubator-daffodil/pull/12#discussion_r158168659
##########
File path:
daffodil-runtime1/src/main/scala/edu/illinois/ncsa/daffodil/debugger/InteractiveDebugger.scala
##########
@@ -1078,6 +1078,35 @@ class InteractiveDebugger(runner:
InteractiveDebuggerRunner, eCompilers: Express
// case e: ExpressionEvaluationException => println(e)
// case e: InfosetException => println(e)
// case e: VariableException => println(e)
+
+ //
+ // mbeckerle: I am unsure why this hack is needed now, but was not
before.
+ // Now if we eval(.) on a node that has no value, we get a RSDE
thrown.
+ // I see no code before (or now) to catch this RSDE, to not create
this
Review comment:
Clarlfiy or now (up untl I put this hack in)
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
