search

mwatts15 / Crono

Scheme project for CS345
0 stars 0 forks source link

Convert let expressions to lambda expressions in Parser.jj #2

Closed mwatts15 closed 11 years ago

mwatts15 commented 11 years ago

Not sure about this one. This happens inside of CronoFunctions.java, and IMHO it's better to do it there than changing our parse to rearrange concrete syntax. Unless there's any objections, I'll close this one.

cvidal commented 11 years ago

I agree, we should keep the parser as compact as possible. The let -> lambda conversion should probably be an intermediate step that we can show with a runtime flag or something since Cannata is probably going to want to see that we did it.

mwatts15 commented 11 years ago

You mean similar to how crono prints out "Environment:" we would have "Converting let expression to function application"?

cvidal commented 11 years ago

Sounds like a good compromise.

mwatts15 commented 11 years ago

I'll push the change later.