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

Fixes overflow when reading in large images #7677

Merged
merged 2 commits into from
Sep 7, 2023
Merged

Fixes overflow when reading in large images #7677

merged 2 commits into from
Sep 7, 2023

Conversation

rdengate
Copy link

Please see #7664

@rdengate rdengate mentioned this pull request Aug 29, 2023
Closed
@lrineau lrineau added this to the 5.6.1 milestone Aug 29, 2023
@lrineau lrineau linked an issue Aug 29, 2023 that may be closed by this pull request
Size overflow for large images #7664
Closed
@lrineau
Copy link
Member

lrineau commented Aug 29, 2023

Thanks for the pull-request. I have added std:: in commit 3527e8e. This pull-request will now be tested, and merged once the tests are fine.

@sloriot
Copy link
Member

sloriot commented Sep 6, 2023

Successfully tested in CGAL-6.0-Ic-56

@lrineau lrineau added the rm only: ready for master For the release team only: that indicates that a PR is about to be merged in 'master' label Sep 6, 2023
@lrineau lrineau assigned lrineau and unassigned sloriot Sep 6, 2023
@lrineau lrineau added the rm only: ready for release branch For the release team only: that indicates that a PR is about to be merged in a release branch label Sep 6, 2023
lrineau added a commit that referenced this pull request Sep 7, 2023
@lrineau
Merge pull request #7677 from rdengate/size_overflow_imread
0803718 
Fixes overflow when reading in large images
@lrineau lrineau merged commit f58e70a into CGAL:5.6.x-branch Sep 7, 2023
8 checks passed
@lrineau lrineau removed rm only: ready for release branch For the release team only: that indicates that a PR is about to be merged in a release branch rm only: ready for master For the release team only: that indicates that a PR is about to be merged in 'master' labels Sep 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@lrineau lrineau

Labels
Bug Merged_in_5.6.1 Merged_in_5.6.2 Merged_in_6.0 Pkg::CGAL_ImageIO Tested
Projects
None yet
Milestone
5.6.1
Development

Successfully merging this pull request may close these issues.

Size overflow for large images
3 participants
@rdengate @lrineau @sloriot