zydeco-lang/zydeco

Error Messages should be in terms of Lines

maxsnew opened this issue · 3 comments

Right now the errors use source locations based on character number. It would be preferable to report this in terms of line/column numbers.

In the compilers course I just mapped over the syntax once it was parsed to rewrite the span1s as span2s. Alternatively we could just keep the line information around and change it when we display the error messages (though because this depends on some external state, it might be difficult to use the Display trait).

We can start implementing Ann struct that keeps all information around once we merge all current work.

Fixed in #35?

Yep : )