stevedlawrence commented on PR #918: URL: https://github.com/apache/daffodil/pull/918#issuecomment-1396930284
> I don't think the spec makes it clear that lengthUnit="bits" cannot be used with xs:integer and xs:nonNegativeInteger. I see the following table: Section 12.3 of the says this in the table describing lengthUnits="bits": > 'bits' may only be used for xs:boolean, xs:byte, xs:short, xs:int, xs:long, xs:unsignedByte, xs:unsignedShort, xs:unsignedInt, and xs:unsignedLong simple types with binary representation, and for calendar (date and time) simple types with binary packed representation. I don't understand the reasoning, bit it is pretty unambiguous. Note that this also adds that it's not allowed for `representation="binary"`. So we *can* have `lengthUnits="bits"` for these types when `representation="text"`, we should ensure we still allow that. -- 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]
