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

Update allocation functions #636

Merged
merged 11 commits into from
Oct 18, 2015
Merged

Update allocation functions #636

merged 11 commits into from
Oct 18, 2015

Conversation

mayeut
Copy link
Collaborator

@mayeut mayeut commented Oct 17, 2015

No description provided.

mayeut added a commit that referenced this pull request Oct 18, 2015
@mayeut
Merge pull request #636 from uclouvain/opj_malloc-625
05b3afd 
Update allocation functions
Fix #625 
Fix #624
Fix #635
@mayeut mayeut merged commit 05b3afd into master Oct 18, 2015
@mayeut mayeut deleted the opj_malloc-625 branch October 18, 2015 01:15
@malaterre
Copy link
Collaborator

@mayeut could you please add some more details on your behavior change for aligned_realloc ?

@mayeut
Copy link
Collaborator Author

mayeut commented Oct 19, 2015

@malaterre, not much to be said.
alignment wasn't check (needs to be power of 2)
a_ptr wasn't check either (if NULL, a crash would have happen)

The main thing is that the aligned_realloc is not used anymore (no need to, an aligned_free + aligned_malloc is all that's needed).

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.
3 participants
@mayeut @malaterre @detonin