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 GL stencil state #500

Merged
merged 1 commit into from Oct 30, 2016
Merged

Fix GL stencil state #500

merged 1 commit into from Oct 30, 2016

Conversation

@kvark
Copy link
Member

kvark commented Oct 28, 2016

This was a typo from #480
Found out by qApitrace log:

major api error 14: GL_INVALID_ENUM in glDisable(GL_STENCIL)

Perhaps, we should check for gl errors in the code too. We seem to only do that for WebGL commands at this point.


This change is Reviewable

@glennw
Copy link
Member

glennw commented Oct 30, 2016

@bors-servo
Copy link
Contributor

bors-servo commented Oct 30, 2016

📌 Commit 42ea22e has been approved by glennw

@bors-servo
Copy link
Contributor

bors-servo commented Oct 30, 2016

Testing commit 42ea22e with merge 34dd22c...

bors-servo added a commit that referenced this pull request Oct 30, 2016
Fix GL stencil state

This was a typo from #480
Found out by qApitrace log:
> major api error 14: GL_INVALID_ENUM in glDisable(GL_STENCIL)

Perhaps, we should check for gl errors in the code too. We seem to only do that for WebGL commands at this point.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/webrender/500)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Oct 30, 2016

☀️ Test successful - status-travis

@bors-servo bors-servo merged commit 42ea22e into master Oct 30, 2016
4 of 6 checks passed
4 of 6 checks passed
continuous-integration/appveyor/branch AppVeyor build failed
Details
continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
dependency-ci Dependencies checked
Details
homu Test successful
Details
@kvark kvark deleted the kvark-stencil branch Oct 31, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.