search

Andriamanitra / coctus

Command line tool for playing clashes and I/O puzzles from CodinGame locally
MIT License
4 stars 2 forks source link

Refactor updating of var_type for JoinTerms #59

Closed daxida closed 5 months ago

daxida commented 5 months ago

Trying to make the updating of JoinTerms a bit more reasonable. The goal is to eventually get rid of is_variable.

I had to make read_pairings pub to deal with a test that (rightfully) crashes now (another option would be to remove the test).

ellnix commented 5 months ago

I like caching the read_pairings, and I like Parser's parse being simple again.

Removing is_variable is a welcome change.