Fix seqof/setof when optional

indutny / asn1.js

ASN.1 Decoder/Encoder/DSL

MIT License

184 stars 64 forks source link

Fix seqof/setof when optional #62

Closed felix closed 8 years ago

felix commented 8 years ago

When seqof() is used with optional() the decoded results are not parsed or included in the resulting object. PR #61 updates _peekTag() to check for seqof/setof when optional. The added tests highlight the issue.

indutny commented 8 years ago

What is this issue for?

felix commented 8 years ago

Opps, sorry @indutny, updated issue description now.

felix commented 8 years ago

Do you need more details @indutny ?

indutny commented 8 years ago

@felix sorry for delay. Landed that patch! Should this issue be closed now?