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

stb_vorbis #defines BUFFER_SIZE which may conflict with other code #1076

Closed
sagamusix opened this issue Jan 15, 2021 · 2 comments
Closed

stb_vorbis #defines BUFFER_SIZE which may conflict with other code #1076

sagamusix opened this issue Jan 15, 2021 · 2 comments
Labels
1 stb_vorbis 5 merged-dev Merged into development branch code quality

Comments

@sagamusix
Copy link
Contributor

In two locations, stb_vorbis #defines a macro called BUFFER_SIZE. This is a very generic name that may conflict with other surrounding code. Can this macro be renamed to STB_BUFFER_SIZE, and maybe also be #undefd afterwards?

@rygorous
Copy link
Collaborator

We will probably merge this, but note that stb_vorbis (unlike most other stb libs) is not a header-file library and is meant to be compiled as its own translation unit; #including it into other code is not an intended use case to begin with.

@rygorous
Copy link
Collaborator

rygorous commented Jul 4, 2021

Fix is merged into dev branch, will be in the next release.

@rygorous rygorous added the 5 merged-dev Merged into development branch label Jul 4, 2021
kcgen pushed a commit to dosbox-staging/dosbox-staging that referenced this issue Jul 13, 2021
kcgen pushed a commit to dosbox-staging/dosbox-staging that referenced this issue Jul 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1 stb_vorbis 5 merged-dev Merged into development branch code quality
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants