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

More renderer debug logs #1331

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

More renderer debug logs #1331

wants to merge 2 commits into from

Conversation

illwieckz
Copy link
Member

Some renderer debug logs I needed to debug things:

  • sdl_glimp: tell if used extension is required or optional

When it is suspected an extension may be buggy, it is useful to know which ones are optional so one can disable them to verify the assumption.

  • renderer: add more debug log about image creation

When the OpenGL driver reports errors when creating or uploading an image, it's useful to have such debug logs to know exactly which image failed, just by reading the immediate previous line. It is useful to identify which image format may be broken in an OpenGL driver.

Copy link
Contributor

@VReaperV VReaperV left a comment

Choose a reason for hiding this comment

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

LGTM.

logger.WithoutSuppression().Notice( "...using GL_%s", name );
if ( flags & ExtFlag_REQUIRED )
{
logger.WithoutSuppression().Notice( "...using required core extension GL_%s.", name );
Copy link
Member

Choose a reason for hiding this comment

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

The message is wrong, taking this branch doesn't mean it must be core

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants