ion-python icon indicating copy to clipboard operation
ion-python copied to clipboard

Handle nop pads correctly

Open raganhan opened this issue 5 years ago • 0 comments

Nop pads cannot be annotated:

  • https://github.com/amzn/ion-tests/blob/master/iontestdata/bad/nopPadWithAnnotations.10n

They are valid inside structs:

  • https://github.com/amzn/ion-tests/blob/master/iontestdata/good/nopPadInsideEmptyStructNonZeroSymbolId.10n
  • https://github.com/amzn/ion-tests/blob/master/iontestdata/good/nopPadInsideStructWithNopPadThenValueNonZeroSymbolId.10n

Relevant spec excerpt:

NOP padding is valid anywhere a value can be encoded, except for within an annotation wrapper. NOP padding in struct requires additional encoding considerations.

raganhan avatar Jan 30 '20 18:01 raganhan