search

heinrich5991 / libtw2

Some Teeworlds stuff in Rust.™
Apache License 2.0
50 stars 16 forks source link

ddnet: Introduce ddnet `DemoWriter` #86

Closed Patiga closed 9 months ago

Patiga commented 10 months ago

Also add --ddnet option to demo_read_write tool.

It automatically writes deltas and only full snapshots every 5 seconds. Delta timings verified with a random demo.

Patiga commented 10 months ago

If the dependency is a blocking concern for a future release on crates.io, then I'll fix it. Alternatively, maybe gamenet_ddnet could be made less version braking? #[non_exhaustive] could be added to enums, but we are missing a Default implementation on structs to use it there.

I myself am only interested in DDNet right now, so tried to get that to work, so that others can skip that work. And yea, I do appreciate simple interfaces. Without it being blocking, I currently lack motivation to make that change.

heinrich5991 commented 9 months ago

Alternatively, maybe gamenet_ddnet could be made less version braking?

Probably not, no. The protocol is going to change in ways that are incompatible for this crate.

Patiga commented 9 months ago

In case of no blocking concerns, could this please get merged?