Dmytro Meleshko

Results 24 comments of Dmytro Meleshko

Same problem on my end, and I believe this is a indeed duplicate of #289

> I think it should not overwrite the source defined `is_available`. cmp should check both. If either one returns `false`, the source is not available. What about passing plugin's original...

@Hopson97 So, the `GameState` trait has two methods: `process_input` and `update`. Both of them return `ReturnResult`. But: when we handle these results in the main loop, some things become uncertain:...

@Hopson97 No, the idea is that game can't do anything before the user inputs a command, so `update` won't be called without `process_input`, therefore they're tightly coupled. Also, look at...

@Hopson97 [Well, well, well...](https://twitter.com/ken_wheeler/status/1022338158393012224?s=09) Maybe, `handle_command`? Or `handle_input`? (Personally, I like the first one) BTW, here's a [PR with these changes](https://github.com/Hopson97/Asciimon/pull/9).

@Hopson97 Just a small request (i don't want to open an issue): could you format the code with `rustfmt`, please?

@antonte What branch/commit do you use? Can you show `Cargo.lock` inside your copy of the project?

@antonte Well, I'm on master, I have exactly the same lock file and I don't get this error. Try updating rust toolchain (`rustup update`) or removing `Cargo.lock`.

What error are you getting? Have you compiled the source code? What have you already tried?