nanopass-framework-racket icon indicating copy to clipboard operation
nanopass-framework-racket copied to clipboard

Published 20 hours ago • verified publisher icon nanopass

define-unparser not provided externally

Open schuster opened this issue 9 years ago • 3 comments

Judging from a quick look through the code, it looks like define-parser and define-unparser are the primary ways to parse and unparse languages. The main file provides define-parser, but not define-unparser. Any particular reason why not both?

schuster avatar Mar 04 '16 20:03 schuster

Oh, hmm, apparently define-language still defines the unparser named after its defined language, but not the parser. That asymmetry is confusing.

schuster avatar Mar 04 '16 20:03 schuster

I agree. @akeep Do you think it would make sense to make it consistent?

LeifAndersen avatar Mar 10 '16 21:03 LeifAndersen

Yes, absolutely.  In fact, I thought it already was, though it appears the not in the Racket version.

-andy:)

On March 10, 2016 at 4:51:34 PM, Leif Andersen ([email protected]) wrote:

I agree. @akeep Do you think it would make sense to make it consistent?

— Reply to this email directly or view it on GitHub.

akeep avatar Mar 11 '16 03:03 akeep