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]

Reply via email to