[ 
https://issues.apache.org/jira/browse/DAFFODIL-2882?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Steve Lawrence resolved DAFFODIL-2882.
--------------------------------------
    Fix Version/s: 3.7.0
       Resolution: Fixed

Fixed in commit 6edf5304a355991dba169b163f2f191a981a1f01

> Incorrect mapping from separatorPoliciy to separatorSuppressionPolicy
> ---------------------------------------------------------------------
>
>                 Key: DAFFODIL-2882
>                 URL: https://issues.apache.org/jira/browse/DAFFODIL-2882
>             Project: Daffodil
>          Issue Type: Bug
>          Components: Front End
>    Affects Versions: 3.6.0
>            Reporter: Steve Lawrence
>            Priority: Major
>             Fix For: 3.7.0
>
>
> Daffodil allows use of the deprecated separatorPolicy property and internally 
> maps it to the new separatorSupressionPolicy property. This is done here:
> https://github.com/apache/daffodil/blob/main/daffodil-lib/src/main/scala/org/apache/daffodil/lib/schema/annotation/props/ByHandMixins.scala#L198-L201
> So our mappings are:
> {code}
> required -> never
> suppressed  -> anyEmpty
> suppressedAtEndStrict -> trailingEmpty
> suppressedAtEndLax -> trailingEmptyStrict 
> {code}
> However, according to the errata document 
> (https://ogf.org/documents/GFD.214.pdf), the mappings should be:
> {code}
> required -> never
> suppressed -> anyEmpty
> suppressedAtEndLax -> trailingEmpty
> suppressedAtEndStrict -> trailingEmptyStrict
> {code}
> Note that suppressedAtEndStrict and suppressedAtEndLax are swapped and do not 
> match the errata document. Comparing the wording of older  versions of the 
> DFDL spec, it does seem like our mapping is incorrect.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to