mbeckerle commented on a change in pull request #61: Preliminary Review - Base64 layering runs first parsing unit test. URL: https://github.com/apache/incubator-daffodil/pull/61#discussion_r182922848
########## File path: daffodil-io/src/main/scala/org/apache/daffodil/processors/charset/USASCII6BitPackedDecoder.scala ########## @@ -26,15 +26,26 @@ import org.apache.daffodil.schema.annotation.props.gen.BitOrder * code unit. * */ - -object USASCII6BitPackedCharset - extends NBitsWidth_BitsCharset("X-DFDL-US-ASCII-6-BIT-PACKED", +private[charset] class USASCII6BitPacked(bitOrder: BitOrder, override val name: String) + extends NBitsWidth_BitsCharset(name, """@ABCDEFGHIJKLMNOPQRSTUVWXYZ[\]^_ !"#$%&'()*+,-./0123456789:;<=>?""", 6, // width - BitOrder.LeastSignificantBitFirst, + bitOrder, 0x1F) { // replacement charCode for encoding of unmapped chars. // // Note: formula for computing string above is // def decodeString = ((0 to 63).map { charCode => (if (charCode <= 31) charCode + 64 else charCode).toChar }).mkString } +object USASCII6BitPackedLSBFirstCharset + extends USASCII6BitPacked(BitOrder.LeastSignificantBitFirst, + "X-DFDL-US-ASCII-6-BIT-PACKED-LSB-FIRST") + +object USASCII6BitPackedCharset + extends USASCII6BitPacked(BitOrder.LeastSignificantBitFirst, + "X-DFDL-US-ASCII-6-BIT-PACKED") // just an alias for the the "LSB-FIRST variant. Review comment: See https://issues.apache.org/jira/browse/DAFFODIL-1926 ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org With regards, Apache Git Services