Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Improved snake example #231

Merged
merged 2 commits into from
Sep 18, 2019
Merged

Improved snake example #231

merged 2 commits into from
Sep 18, 2019
+592 −261

Conversation

zrzka
Copy link
Contributor

@zrzka zrzka commented Sep 18, 2019

Tested on the macOS only.

Fixes #230.

@zrzka
Improved snake example
231bc8c 
Signed-off-by: Robert Vojta <rvojta@me.com>
TimonPost
TimonPost approved these changes Sep 18, 2019
View reviewed changes
Copy link
Member

@TimonPost TimonPost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, I saw that it worked. I will give it a try my self for the next release. I am not going to review all the code, I looked it trough, but it doesn't really matter to me if there is a comment or unwrap left somewhere.

examples/program_examples/snake/src/main.rs Outdated Show resolved Hide resolved
examples/program_examples/snake/src/messages.rs Show resolved Hide resolved
@zrzka
Fix typo
b3493c6 
Signed-off-by: Robert Vojta <rvojta@me.com>
@TimonPost TimonPost merged commit 91d0727 into crossterm-rs:master Sep 18, 2019
@zrzka zrzka deleted the zrzka/snake-example-fixes branch September 19, 2019 11:56
december1981 pushed a commit to december1981/crossterm that referenced this pull request Oct 26, 2023
@patrickfreed
RUST-870 Support deserializing directly from raw BSON (crossterm-rs#276)
7ccf82b 
This commit also adds support for borrowed deserialization (crossterm-rs#231, RUST-688) and fixes RUST-880 and RUST-884.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@TimonPost TimonPost TimonPost approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Snake example on macOS
2 participants
@zrzka @TimonPost