-
Notifications
You must be signed in to change notification settings - Fork 247
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
New shots for Solo #416
New shots for Solo #416
Conversation
} | ||
|
||
SoloPanoOptions(ByteBuffer buffer) { | ||
this(buffer.get(), buffer.get() == PANO_ON_VALUE, (short) buffer.getShort(), buffer.getFloat(), buffer.getFloat()); |
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.
@phmagic the first parameter is an int
, so buffer.getInt()
should be used instead.
buffer.getShort()
doesn't need a cast as well.
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.
@phmagic nevermind, I see below you're storing the panoPreference
as a byte.
@phmagic In addition, can you add a javadoc on top of the new |
- Pano and Rewind options: restrict input of preference parameters - SoloZiplineLock - removed necessary code - build.gradle - revert back to 1.5.0
- Made PanoOptions a subclass of TLVPacket since it does not use anything from the previous parent (SoloShotOptions) - changed PanoStatus variables to byte instead of int
@phmagic looks good to go. I'll merge this pr in, feel free to open another one with the updated javadocs. |
Zipline Message
Pano Messages
Rewind Options Message
Return to Me Message