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.

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

requirement for status list size #47

Closed
paulbastian opened this issue Jul 18, 2023 · 3 comments · Fixed by #172
Closed

requirement for status list size #47

paulbastian opened this issue Jul 18, 2023 · 3 comments · Fixed by #172
Assignees

Comments

@paulbastian
Copy link
Contributor

As we declined to include a size property to the status list, we do not have other means than to decode&decompress the encoded status list and get status list size from the byte array length/size:
status list size = byte array size * 8 / bits
Therefore it makes to add the requirement that status list size must "fill up" to bytes

@paulbastian
Copy link
Contributor Author

ready for PR after #57

@paulbastian
Copy link
Contributor Author

Editors Call:

  • only danger that we currently see is that some bits at the end have random/un-assigned values
  • leaving this open for now

@paulbastian
Copy link
Contributor Author

Editors Call:

  • maybe add this as a recommendation in implementation consideration

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant