-
Notifications
You must be signed in to change notification settings - Fork 35
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.
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
Fix build warnings for examples #37
Conversation
On
|
This is a bit tricky. I'm not used to assigning a value while evaluating a condition (in C++, rust has better syntax for that). I think the requested fix would involve setting |
Yup. How about something like |
That's actually kinda redundant, |
Hey, I'm still waking up, realized that as soon as I posted it. |
I'm not actually setup to hardware test this lib. Have you run this with the latest changes (as requested)? |
Not yet, likely sometime today if all goes as planned. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good to go!
resolves #35