[GitHub] [daffodil] mike-mcgann commented on pull request #966: Better error message when contentLength/valueLength argument is an array

["mike-mcgann (via GitHub)" <gi...@apache.org>]

mike-mcgann commented on PR #966:
URL: https://github.com/apache/daffodil/pull/966#issuecomment-1438630902

   > I think the change is simple though. Add `def isSelf: Boolean` to PathExpression, and it is true if the path is just the Self step alone. Then use that to test for the exception, not the type.
   
   Updated. 
    
   > The hexBinary test should then still work, but we should also add a test with a small simple complex type to show this working for that case also.
   
   It looks like test [valueLength11](https://github.com/apache/daffodil/blob/main/daffodil-test/src/test/resources/org/apache/daffodil/section23/dfdl_functions/Functions.tdml#L418-L435) covers that condition. 
   


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