Skip to content

Handle GL_MAX_VIEWS_OVR in glGetInteger#1583

Merged
pmuetschard merged 1 commit into
google:masterfrom
pmuetschard:ovr
Mar 2, 2018
Merged

Handle GL_MAX_VIEWS_OVR in glGetInteger#1583
pmuetschard merged 1 commit into
google:masterfrom
pmuetschard:ovr

Conversation

@pmuetschard

Copy link
Copy Markdown
Member

No description provided.

Comment thread gapis/api/gles/api/state_queries.api Outdated
}
@if(Extension.GL_OVR_multiview)
case GL_MAX_VIEWS_OVR: {
write(v[0:1])

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If we don't have a constant for this that can be assigned out, please add a TODO to add one.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've added the constant. PTAL.

Comment thread gapis/api/gles/helpers.go
MaxTransformFeedbackSeparateAttribs: 4,
MaxTransformFeedbackSeparateComponents: 4,
MaxTextureMaxAnisotropyExt: 2.0,
MaxViewsExt: 2,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe this should default to 1?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, the minimum is 2.

@pmuetschard pmuetschard merged commit b45397b into google:master Mar 2, 2018
@pmuetschard pmuetschard deleted the ovr branch March 2, 2018 20:19
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.

2 participants