mflatt
2017-2-23 15:50:24

I’ve now expanded my first macro using the Racket expander running on Chez!


samth
2017-2-23 15:52:04

woo!


samth
2017-2-23 15:52:42

do you have a sense for why the change to not generate let-values made such a difference, even though chez expands let-values in almost exactly the same way?


mflatt
2017-2-23 15:53:44

My guess is that something is wrong with the let-values expansion in Chez – accidental quadratic behavior, maybe – but I haven’t investigated


samth
2017-2-23 15:58:33

I can now (with the contents of my latest PR) run #lang racket programs


samth
2017-2-23 16:05:57

ah darn, still not quite


samth
2017-2-23 16:12:12

@mflatt the strategy for literal regexps in convert.rkt doesn’t work for '(a b c #rx"") and a repair is not obvious


samth
2017-2-23 16:12:51

if you have a suggestion for how to do it, I’ll take a stab at implementing


mflatt
2017-2-23 16:13:00

You understand that “convert.rkt” is a stop-gap that’s intended for just the expander and layers that live below it, right?


mflatt
2017-2-23 16:13:21

I’m ok with experiments running other Racket modules via “convert.rkt”, but I don’t think we want to invest too much there


samth
2017-2-23 16:14:41

The longer-term strategy is to directly use read as extracted, right?


samth
2017-2-23 16:14:53

as well as expand


mflatt
2017-2-23 16:15:22

Right – everything will be loaded and run by the implementation in “expand.sls”


mflatt
2017-2-23 16:16:06

If you want to keep playing with uses of “convert.rkt”, though, I’m fine with that – and I don’t plan to reorganize things again for a few hours, if you’d like to resync the PR


samth
2017-2-23 17:59:16

@mflatt my pull request is now re-synced, and also contains some useful fixes (demoed in <http://expander-demo.ss\|expander-demo.ss>)


samth
2017-2-23 18:00:45

it relies on a small bug fix in the linklet branch, which you can find in a PR there


mflatt
2017-2-23 18:19:48

Merged, but I’d like to revert “Enable print-graph”, since the intent is to not use the Chez printer


samth
2017-2-23 18:21:31

Sure, that makes sense


mflatt
2017-2-23 18:22:03

@samth When I try to build the with the linlket commits, the math package fails with a TR error; is that because the expansion of in-... changed, or something else?


samth
2017-2-23 18:22:20

that’s probably why, I’ll investigate


samth
2017-2-24 00:00:36

@mflatt the TR/math issue is fixed by the combination of https://github.com/mflatt/racket/pull/13 and https://github.com/racket/typed-racket/pull/501


samth
2017-2-24 00:00:50

I’ll merge the TR one once the CI finishes there


samth
2017-2-24 00:34:04

now merged


samth
2017-2-24 04:13:04

I wrote a repl, but somehow it doesn’t work when I define identifiers


samth
2017-2-24 04:15:35