search

dkg / draft-rivest-sexp

0 stars 0 forks source link

ABNF confusion #1

Open dkg opened 1 year ago

dkg commented 1 year ago

The ABNF in this document has three instances that define the sexp node (for "basic", "canonical", and "advanced" forms).

These re-definitions each repeat some sub-nodes (e.g. raw and decimal), rather than reusing existing definitions.

I like the split out of the different forms, but i think they should be named independently, and each one should build on the previous definitions rather than repeating them.

d3e3e3 commented 5 months ago

I mostly agree with you. I'm not sure identical redundant rules between the different representations are a problem but there should not be different definitions of "xexp".

d3e3e3 commented 5 months ago

The ABNF has been completely reformulated and improved.