You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@daffodil.apache.org by "Mike Beckerle (Jira)" <ji...@apache.org> on 2020/07/30 21:49:00 UTC

[jira] [Commented] (DAFFODIL-845) textNumberProps: Issues with textNumberCheckPolicy="lax"

    [ https://issues.apache.org/jira/browse/DAFFODIL-845?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17168244#comment-17168244 ] 

Mike Beckerle commented on DAFFODIL-845:
----------------------------------------

lax behavior is basically not well defined by the ICU library and the maintaners of that insist that "lax" behavior can't be pinned down (per email threads on DFDL workgroup email list). 

DFDL spec is evolving to be less specific about what "lax" means for number parsing. 

> textNumberProps: Issues with textNumberCheckPolicy="lax"
> --------------------------------------------------------
>
>                 Key: DAFFODIL-845
>                 URL: https://issues.apache.org/jira/browse/DAFFODIL-845
>             Project: Daffodil
>          Issue Type: Bug
>          Components: ICU
>    Affects Versions: s11
>            Reporter: Jessie Chab
>            Priority: Major
>
> According to errata item 2.21:
> 2.21. Section 13.6. Change meaning of textNumberCheckPolicy enum 'lax' to align with ICU.: "If ���lax' and dfdl:textNumberRep is 'standard' then grouping separators are ignored, leading and trailing whitespace is ignored, leading zeros are ignored, quoted characters may be omitted."
> I wrote two tests for the last part of that requirement, and they result in Parse Errors.
> see tests test_textNumberCheckPolicy_lax01, test_textNumberCheckPolicy_lax05
> in files:
> daffodil-test/src/test/resources/edu/illinois/ncsa/daffodil/section13/text_number_props/TextNumberProps.tdml
> daffodil-test/src/test/scala-debug/edu/illinois/ncsa/daffodil/section13/text_number_props/TestTextNumberPropsDebug.scala
> daffodil-test/src/test/scala/edu/illinois/ncsa/daffodil/section13/text_number_props/TestTextNumberProps.scala



--
This message was sent by Atlassian Jira
(v8.3.4#803005)