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 2024/11/18 16:08:00 UTC

[jira] [Commented] (DAFFODIL-2957) dfdl:decimalSigned="no" bugs with negative values and data representations

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

Mike Beckerle commented on DAFFODIL-2957:
-----------------------------------------

This basically doesn't work well, and has no test coverage. 

> dfdl:decimalSigned="no" bugs with negative values and data representations
> --------------------------------------------------------------------------
>
>                 Key: DAFFODIL-2957
>                 URL: https://issues.apache.org/jira/browse/DAFFODIL-2957
>             Project: Daffodil
>          Issue Type: Bug
>          Components: Back End
>    Affects Versions: 3.9.0
>            Reporter: Mike Beckerle
>            Assignee: Mike Beckerle
>            Priority: Minor
>
> Tests are needed for whether unparsing a negative value as an element of type xs:decimal with dfdl:decimalSigned="no" detects an error or not. 
> When dfdl:binaryNumberRep="bcd" this is definitely an error and should produce an Unparse error. (IBM DFDL does detect this.)
> When dfdl:binaryNumberRep is 'binary', 'packed', or 'ibm4790Packed', the data stream can represent a negative sign, the question is whether this error is detected or not. 
> The DFDL Workgroup wants all implementations to test if this error is detected currently because if this was clarified in the spec as causing an unparse error, but current implementations are not detecting this currently, then this would be a breaking change that would require more careful analysis, and possibly switches to provide backward compatibility. Or the behavior would just become "implementation defined". 
>  



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