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

Max Correlation Id should be INT32_MAX, not UINT16_MAX #1214

Closed
joeljfischer opened this issue Apr 4, 2019 · 0 comments
Closed

Max Correlation Id should be INT32_MAX, not UINT16_MAX #1214

joeljfischer opened this issue Apr 4, 2019 · 0 comments
Assignees
@joeljfischer
Labels
bug A defect in the library
Milestone
6.2.0

Comments

@joeljfischer
Copy link
Contributor

Bug Report

According to Core and the updated protocol spec, the maximum value of a correlation Id is a signed 32-bit integer. The library currently uses an unsigned 16-bit integer. We should up our max correlation id to match the spec.

OS & Version Information
  • iOS Version: n/a
  • SDL iOS Version: 6.2-develop
  • Testing Against: n/a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@joeljfischer joeljfischer

Labels
bug A defect in the library
Projects
None yet
Milestone
6.2.0
Development

No branches or pull requests
1 participant
@joeljfischer