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_r182921926
########## File path: daffodil-io/src/main/scala/org/apache/daffodil/processors/charset/AIS_PAYLOAD_ARMORING.scala ########## @@ -0,0 +1,28 @@ +package org.apache.daffodil.processors.charset + +import org.apache.daffodil.schema.annotation.props.gen.BitOrder + +/** + * Special purpose. This is not used for decoding anything. + * The encoder is used to convert strings using the characters + * allowed, into binary data using the AIS Payload Armoring + * described here: + * + * http://catb.org/gpsd/AIVDM.html#_aivdm_aivdo_payload_armoring + * + * To convert a string of length N, You will get 6N bits. + * + * The decoder can be used for unit testing, but the point of this class + * is to make the encoder available for use in un-doing the AIS Payload + * armoring. + * + * Always use with encodingErrorPolicy="error" since supplying a character Review comment: I'll improve the comment. We don't have the EEP='error' option, so wherever this AIS encoder gets called we'll need to do the checking ourselves. ---------------------------------------------------------------- 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