Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

Fix tile debug rendering #7373

Merged
merged 1 commit into from
Dec 13, 2016
Merged

Fix tile debug rendering #7373

merged 1 commit into from
Dec 13, 2016

Conversation

jfirebaugh
Copy link
Contributor

@jfirebaugh jfirebaugh commented Dec 10, 2016

Regressed in #7233.

  • Off-by-one in DebugBucket indexing
  • Must unbind VAO before binding index buffer in Context::createIndexBuffer
  • Get test-suite test passing so we catch this sort of regression

@mention-bot
Copy link

@jfirebaugh, thanks for your PR! By analyzing this pull request, we identified @ansis, @tmpsantos and @yhahn to be potential reviewers.

@jfirebaugh jfirebaugh requested a review from tmpsantos December 10, 2016 01:55
@jfirebaugh
Copy link
Contributor Author

sigh Rendering still fails in CI. GL_LINE just varies too much between implementations / graphics cards.

* Off-by-one in DebugBucket indexing
* Must unbind VAO before binding index buffer in Context::createIndexBuffer
@jfirebaugh
Copy link
Contributor Author

Abandoned the test changes.

Copy link
Member

@brunoabinader brunoabinader left a comment

Choose a reason for hiding this comment

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

Tested on macOS 👍

@jfirebaugh jfirebaugh merged commit c12b52c into master Dec 13, 2016
@jfirebaugh jfirebaugh deleted the fix-debug branch December 13, 2016 17:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants