IGC file writer about igc-rs HOT 8 CLOSED

@Turbo87 - I think it's more idiomatic to make it impossible to create invalid record structs - every sane instance of a record struct has a single well defined string representation.

@estk - I started having a look at implementing this over the weekend, I have just opened a WIP PR (#15) to track it.

from igc-rs.

I hadn't thought of that yet, but you're right that it would be fairly straightforward to add. I'm happy to do so, though feel free to open a PR :)

from igc-rs.

@estk do you want to look into that?

from igc-rs.

On further thought, implementing formatters for the various record types will actually help with adding property testing, as it enables property tests of the form format!("{}", rec).parse::<Record>() == rec.

from igc-rs.

yep, although I'm wondering if the formatters should return Result to in case of invalid values 🤔

from igc-rs.

@Turbo87 @Joey9801 I'd be happy to look into this, might be a few weeks though.

from igc-rs.

I think it's more idiomatic to make it impossible to create invalid record structs - every sane instance of a record struct has a single well defined string representation.

yeah, that makes sense to me. just wondering how to implement that. are you thinking about a builder pattern kind of thing?

from igc-rs.

Implemented in #15

from igc-rs.