olabusayoT commented on code in PR #1613:
URL: https://github.com/apache/daffodil/pull/1613#discussion_r2728987137
##########
daffodil-test/src/test/resources/org/apache/daffodil/usertests/SepTests.tdml:
##########
@@ -456,11 +456,11 @@
<xs:element name="root" dfdl:initiator="RECORD">
<xs:complexType>
- <xs:sequence dfdl:separator="/" dfdl:separatorPosition="prefix"
dfdl:terminator="%NL;">
+ <xs:sequence dfdl:separator="/" dfdl:separatorPosition="prefix"
dfdl:terminator="%NL;" dfdl:separatorSuppressionPolicy="anyEmpty">
<xs:element name="field1" type="xs:string" />
<xs:element name="field2" type="xs:string" minOccurs="0" />
<xs:element name="field3" type="xs:string" minOccurs="0" />
- <xs:element name="groupOfFields" minOccurs="1" maxOccurs="3"
dfdl:occursCountKind="parsed">
+ <xs:element name="groupOfFields" minOccurs="1" maxOccurs="3">
Review Comment:
I changed both since I think "parsed" was not the right option what with the
min/maxOccurs, so implicit seemed like the better option. I also changed SSP
because "never" would add in the additional prefix separator for the missing
groupOfFields element. Technically nothing needed to change, but OCK=implicit
and SSP=anyEmpty seems like the correct thing based on the attributes and the
data/infoset
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]