-
Notifications
You must be signed in to change notification settings - Fork 37
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # 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