olabusayoT commented on code in PR #1509:
URL: https://github.com/apache/daffodil/pull/1509#discussion_r2216511921


##########
daffodil-core/src/test/scala/org/apache/daffodil/core/dsom/TestMiddleEndAttributes.scala:
##########
@@ -179,7 +179,10 @@ class TestMiddleEndAttributes {
     val mems = seq.groupMembers
     val Seq(t1: Term) = mems
     val e1ref = t1.asInstanceOf[ElementRef]
-    val Some(nes: LocalSequence) = e1ref.optLexicalParent
+    val nes = e1ref.optLexicalParent match {
+      case Some(nes: LocalSequence) => nes
+      case _ => fail(); null
+    }

Review Comment:
   No, I'd discussed option with Steve before he left and unchecked (which is 
the recommended way) didn't seem like the best fit



-- 
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-unsubscr...@daffodil.apache.org

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

Reply via email to