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

Rename public structs and defines to include the prefix CGIF (namespacing) #18

Merged
merged 2 commits into from
Sep 2, 2021

Conversation

dloebl
Copy link
Owner

@dloebl dloebl commented Sep 1, 2021

Add namespacing for CGIF as suggested in #1.
Note: Breaks existing code - OK as no tagged release / version defined yet.

@dloebl dloebl requested a review from MCLoebl September 1, 2021 19:07
@lovell
Copy link
Contributor

lovell commented Sep 1, 2021

Danke Schoen / Thank you very much for this PR. Much better to make this kind of improvement to the API before anyone important starts using it ;)

@dloebl dloebl merged commit 80bd8fb into main Sep 2, 2021
@dloebl dloebl deleted the breaking-namespacing branch September 2, 2021 18:52
lovell added a commit to lovell/libvips that referenced this pull request Sep 3, 2021
jcupitt pushed a commit to libvips/libvips that referenced this pull request Sep 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants