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

Fix-comp-warns #9

Closed
wants to merge 8 commits into from
Closed

Conversation

drewwoods
Copy link

Although snprintf fixes possible overflows, the source strings might be
truncated to fit in the dest. Compiling with -Wno-format-truncation
removes this warning.

Shh compiler, its okay, or is it?

* also support GL_ARB_multitexture for multitexturing

Now both GL_SGIS_multitexture and GL_ARB_multitexture are supported

At the time of release GL_SGIS_multitexture was the multitexturing
extension available.  However GL_ARB_multitexture is the univeral
extension, in fact I don't believe any modern OpenGL implementation
supports GL_SGIS_multitexture.

This had to do some minor rework of the multitexturing code to support
more than one implementation.

* update naming
Although snprintf fixes possible overflows, the source strings might be
truncated to fit in the dest. Compiling with -Wno-format-truncation
removes this warning.

Shh compiler, its okay, or is it?
@drewwoods drewwoods closed this Jun 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant