This extension is also core in OpenGL 4.3 and OpenGL ES 3.2. It lets us associate a string to pretty much any object, push and pop debug groups, and get a callback whenever some part of the stack wants to talk with us.
Details
Details
- Reviewers
- None
- Build in debug mode.
- Test with a debugger such as apitrace
- Textures should now have their original path attached.
- Listing of functions should be much easier to read, as they are grouped by purpose.
- Driver information about performance issues and such should be printed in the terminal.
Diff Detail
Diff Detail
- Repository
- rP 0 A.D. Public Repository
- Branch
- /ps/trunk
- Lint
Lint OK - Unit
No Unit Test Coverage - Build Status
Buildable 10926 Build 19199: Vulcan Build Jenkins Build 19198: Vulcan Build (macOS) Jenkins Build 19197: Vulcan Build (Windows) Jenkins Build 19196: arc lint + arc unit
Event Timeline
Comment Actions
Successful build - Chance fights ever on the side of the prudent.
Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/803/display/redirect
Comment Actions
Successful build - Chance fights ever on the side of the prudent.
Linter detected issues: Executing section Source... source/lib/external_libraries/glext_funcs.h | 1| /*·Copyright·(C)·2013·Wildfire·Games. | | [NORMAL] LicenseYearBear: | | License should have "2019" year instead of "2013" source/lib/res/graphics/ogl_tex.cpp | 1| /*·Copyright·(C)·2010·Wildfire·Games. | | [NORMAL] LicenseYearBear: | | License should have "2019" year instead of "2010" Executing section JS... Executing section cli...
Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1319/display/redirect
Comment Actions
Successful build - Chance fights ever on the side of the prudent.
Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/805/display/redirect
Comment Actions
Successful build - Chance fights ever on the side of the prudent.
Linter detected issues: Executing section Source... source/lib/external_libraries/glext_funcs.h | 1| /*·Copyright·(C)·2013·Wildfire·Games. | | [NORMAL] LicenseYearBear: | | License should have "2019" year instead of "2013" source/lib/res/graphics/ogl_tex.cpp | 1| /*·Copyright·(C)·2010·Wildfire·Games. | | [NORMAL] LicenseYearBear: | | License should have "2019" year instead of "2010" Executing section JS... Executing section cli...
Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1321/display/redirect
Comment Actions
Successful build - Chance fights ever on the side of the prudent.
Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/806/display/redirect
Comment Actions
Successful build - Chance fights ever on the side of the prudent.
Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1322/display/redirect
Comment Actions
- Add debug groups push and pop using RAII.
- Add debug message callback with filtering.
Comment Actions
Build failure - The Moirai have given mortals hearts that can endure.
Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/862/display/redirect
Comment Actions
Build failure - The Moirai have given mortals hearts that can endure.
Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/863/display/redirect
Comment Actions
Successful build - Chance fights ever on the side of the prudent.
Linter detected issues:
Executing section Source...
source/lib/ogl.h
| 1| /*·Copyright·(C)·2017·Wildfire·Games.
| | [NORMAL] LicenseYearBear:
| | License should have "2019" year instead of "2017"
source/lib/ogl.h
| 198| class·ogl_DebugGroup
| | [MAJOR] CPPCheckBear (syntaxError):
| | Code 'classogl_DebugGroup{' is invalid C code. Use --std or --language to configure the language.
source/lib/ogl.cpp
| 1| /*·Copyright·(C)·2017·Wildfire·Games.
| | [NORMAL] LicenseYearBear:
| | License should have "2019" year instead of "2017"
Executing section JS...
Executing section cli...Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1378/display/redirect
Comment Actions
- Update the copyright years.
- Simplify the ogl_DebugGroup class.
- Add a fallback for when we’re not compiling in debug mode.
Comment Actions
Build failure - The Moirai have given mortals hearts that can endure.
Link to build: https://jenkins.wildfiregames.com/job/vs2015-differential/864/display/redirect
Comment Actions
Successful build - Chance fights ever on the side of the prudent.
Linter detected issues:
Executing section Source...
source/lib/ogl.h
| 1| /*·Copyright·(C)·2017·Wildfire·Games.
| | [NORMAL] LicenseYearBear:
| | License should have "2019" year instead of "2017"
source/lib/ogl.h
| 198| class·ogl_DebugGroup
| | [MAJOR] CPPCheckBear (syntaxError):
| | Code 'classogl_DebugGroup{' is invalid C code. Use --std or --language to configure the language.
source/lib/ogl.cpp
| 1| /*·Copyright·(C)·2017·Wildfire·Games.
| | [NORMAL] LicenseYearBear:
| | License should have "2019" year instead of "2017"
Executing section JS...
Executing section cli...Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1379/display/redirect
Comment Actions
Successful build - Chance fights ever on the side of the prudent.
Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1380/display/redirect
| source/lib/external_libraries/glext_funcs.h | ||
|---|---|---|
| 156 | We have increasing number of version, so I think it'd be better to put this section at the end of the #else - #endif section. Like it's done for GLES. | |
| source/lib/ogl.h | ||
| 212 | It doesn't make sense to generate even an empty struct. I suggest to replace ogl_DebugGroup group("Draw GUI element"); by one macro like OGL_DEBUG_GROUP("Draw GUI element");, like it's done for profiler. BTW can't we just add groups in PROFILER2? Because we already have GL queries there and it seems reasonable to have them together. | |
| source/lib/ogl.h | ||
|---|---|---|
| 212 | Note that your PROFILE2() macro does exactly the same thing, by creating a CProfile2Region object named profile2__ (see source/ps/Profiler2.h:536), except this object contains a m_Name pointer so it can’t be eluded by the compiler. It shouldn’t contain it though, you don’t reuse it at all after the constructor is done, that sounds like an easy optimisation. Adding groups the same way would be ok, but I’m not sure why go through the indirection of your profiler when you just want to have raii for OpenGL calls. | |
Comment Actions
Fix windows build. Apitrace doesn't work for me though, it segfaults directly
EDIT: I do get warnings such as these though:
warning: OpenGL other of severity notification from the API id 131185: Buffer detailed info: Buffer object 1 (bound to GL_ELEMENT_ARRAY_BUFFER_ARB, usage hint is GL_STATIC_DRAW) will use VIDEO memory as the source for buffer object operations.
Also lots of warnings for animated meshes (even though they are not moving in atlas) and one or two for static meshes.
Comment Actions
Build failure - The Moirai have given mortals hearts that can endure.
Link to build: https://jenkins.wildfiregames.com/job/macos-differential/212/display/redirect
Comment Actions
Successful build - Chance fights ever on the side of the prudent.
Linter detected issues: Executing section Source... source/gui/ObjectTypes/CMiniMap.cpp | 1| /*·Copyright·(C)·2019·Wildfire·Games. | | [NORMAL] LicenseYearBear: | | License should have "2020" year instead of "2019" source/ps/CLogger.cpp | 1| /*·Copyright·(C)·2019·Wildfire·Games. | | [NORMAL] LicenseYearBear: | | License should have "2020" year instead of "2019" source/renderer/TerrainRenderer.cpp | 1| /*·Copyright·(C)·2019·Wildfire·Games. | | [NORMAL] LicenseYearBear: | | License should have "2020" year instead of "2019" source/lib/ogl.h | 1| /*·Copyright·(C)·2019·Wildfire·Games. | | [NORMAL] LicenseYearBear: | | License should have "2020" year instead of "2019" source/lib/res/graphics/ogl_tex.cpp | 1| /*·Copyright·(C)·2019·Wildfire·Games. | | [NORMAL] LicenseYearBear: | | License should have "2020" year instead of "2019" source/gui/CGUIText.cpp | 1| /*·Copyright·(C)·2019·Wildfire·Games. | | [NORMAL] LicenseYearBear: | | License should have "2020" year instead of "2019" source/gui/GUIRenderer.cpp | 1| /*·Copyright·(C)·2019·Wildfire·Games. | | [NORMAL] LicenseYearBear: | | License should have "2020" year instead of "2019" source/lib/ogl.cpp | 1| /*·Copyright·(C)·2019·Wildfire·Games. | | [NORMAL] LicenseYearBear: | | License should have "2020" year instead of "2019" source/renderer/WaterManager.cpp | 1| /*·Copyright·(C)·2019·Wildfire·Games. | | [NORMAL] LicenseYearBear: | | License should have "2020" year instead of "2019" Executing section JS... Executing section cli...
Link to build: https://jenkins.wildfiregames.com/job/docker-differential/1634/display/redirect
Comment Actions
Just confirmed that the crash in debug mode of apitrace is not linked to this patch. So only thing to fix is Mac Os.