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

Add: A video driver that shows what areas of the screen are dirty. #8220

Closed
wants to merge 1 commit into from

Conversation

@techgeeknz
Copy link
Contributor

techgeeknz commented Jun 13, 2020

This SDL-based video driver can be useful for debugging graphics routines.
It draws a dashed rectangle around all screen areas that have been marked dirty.

This SDL-based video driver can be useful for debugging graphics routines.
It draws a dashed rectangle around all screen areas that have been marked dirty.
@techgeeknz techgeeknz force-pushed the techgeeknz:sdl_dirty branch from c813837 to 3b18613 Jun 13, 2020
@techgeeknz
Copy link
Contributor Author

techgeeknz commented Jun 13, 2020

Huh. The change to CMakeLists.txt somehow vanished when I git stashed 😕

@LordAro
Copy link
Member

LordAro commented Jun 13, 2020

Don't we already have this with "Toggle dirty blocks" in the about menu (or Ctrl+I) , or am I missing something obvious?

@techgeeknz
Copy link
Contributor Author

techgeeknz commented Jun 13, 2020

Don't we already have this with "Toggle dirty blocks" in the about menu (or Ctrl+I) , or am I missing something obvious?

I believe that is intended an end-user debugging feature for NewGRF authors. Specifically, it only marks the redrawn areas of viewports.

This works at a lower level, making it more useful for debugging and/or restructuring the graphics system.

@techgeeknz
Copy link
Contributor Author

techgeeknz commented Jun 15, 2020

There doesn't seem to be a lot of interest in this; and I'm not entirely convinced that it's the best approach to the problem anyway. Closing.

@techgeeknz techgeeknz closed this Jun 15, 2020
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

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