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

context constants / limits #32

Open
zacharias2k opened this issue Sep 20, 2013 · 3 comments
Open

context constants / limits #32

zacharias2k opened this issue Sep 20, 2013 · 3 comments

Comments

@zacharias2k
Copy link

Why are all opengl context constants hardcoded?
Would be nice to set them after construction of the glsl context.

@aras-p
Copy link
Owner

aras-p commented Sep 20, 2013

Do you have a good idea how to expose that?

@zacharias2k
Copy link
Author

I think we should leave the very tiny API as it is, so options are limited.

  • The easiest way would be via preprocessor defines, but that wouldn't be very flexible if anyone wants to deliver binaries for different machines.
  • a function like glslopt_setlimit(glslopt_ctx*, option, value) - most values are integers, so it should be easy to access, but ugly IMHO, because it's not visible to the user and the context shouldn't change over execution time.
  • enhance glslopt_initialize, add some parameters or give it a structure containing the settings.

But i'm not happy with those solutions, you might want to come up with something better.

PS: The reason for this was mainly because of MaxDrawBuffers, which is 2 - far too low, even mesa updated it to 8, years ago, see config.h line 228.

#define MAX_DRAW_BUFFERS 8
You may want to change that too.

@sagaceilo
Copy link

I had the same Issue with MAX_DRAW_BUFFERS , especialy if You using some kind of MRT rendering.

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

No branches or pull requests

3 participants