search

jyn514 / saltwater

A C compiler written in Rust, with a focus on good error messages.
BSD 3-Clause "New" or "Revised" License
294 stars 27 forks source link

Close #494. Update pico-args with short-space-opt feature and use str for paths #497

Closed hdamron17 closed 4 years ago

hdamron17 commented 4 years ago

Close #494. Following pico-args #20, this pull request takes advantage of the short-space-opt feature which makes -Iinclude possible. In order to do this, we had to use str instead of OsStr which isn't ideal.

jyn514 commented 4 years ago

This is making me realize we need tests for CLI arguments :/ I'll test manually so we don't have to block this PR but I think we should add tests at some point.

hdamron17 commented 4 years ago

I was thinking the same thing..