mattwparas / steel

An embedded scheme interpreter in Rust
Apache License 2.0
980 stars 46 forks source link

improve list locations #215

Closed jrvidal closed 1 month ago

jrvidal commented 1 month ago

I noticed we sometimes lack span info for ExprKind::List instances, while I was trying to improve error locations. So this PR does both things.

mattwparas commented 1 month ago

This LGTM - any other changes you wanted to make? Otherwise I can approve and merge

jrvidal commented 1 month ago

Nope, this is everything I had re: list spans.

mattwparas commented 1 month ago

Thanks!