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

Fix more issues in preparation for v1.10 #383

Merged
merged 2 commits into from
Oct 22, 2022
Merged

Fix more issues in preparation for v1.10 #383

merged 2 commits into from
Oct 22, 2022

Conversation

Strilanc
Copy link
Collaborator

@Strilanc Strilanc commented Oct 22, 2022 •
edited
Loading

  • Add fuzz testing of all the read/write shot formats and options
  • Add ability to pass pathlib.Path type to read/write shot methods
  • Fix a bug in the ptb64 writing logic related to non-multiple-of-64 shot sizes
  • Deprecate bit_pack argument name in favor of bit_packed
  • Fix false positive "writingto zero sized region" warning from GCC from std::vector::insert
  • Add a dev script for regenerating the texture data as C++ without exceeding allowed literal sizes
  • Add missing cp311-macosx_x86_64 wheel

Fixes #382

@Strilanc
Fix more issues in preparation for v1.10
182f3a4 
- Add fuzz testing of all the read/write shot formats and options
- Add ability to pass pathlib.Path type to read/write shot methods
- Fix a bug in the ptb64 writing logic related to non-multiple-of-64 shot sizes
- Deprecate `bit_pack` argument name in favor of `bit_packed`
- Fix false positive "writingto zero sized region" warning from GCC from std::vector::insert
- Add a dev script for regenerating the texture data as C++ without exceeding allowed literal sizes
- Add missing cp311-macosx_x86_64 wheel
@Strilanc
Also fix the sparse shot reader
a3f9a08
@Strilanc Strilanc enabled auto-merge (squash) October 22, 2022 21:43
@Strilanc Strilanc merged commit a2379f1 into main Oct 22, 2022
@Strilanc Strilanc deleted the fixliteralsizewarning branch October 22, 2022 21:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

string literal length warning and "AddressSanitizer: container-overflow"
1 participant
@Strilanc