[PR #15249] [MERGED] Don't just die if the user doesn't have the dx debugging tools #30532

Open
opened 2026-01-31 09:41:24 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/microsoft/terminal/pull/15249
Author: @zadjii-msft
Created: 4/27/2023
Status: Merged
Merged: 4/27/2023
Merged by: @DHowett

Base: mainHead: dev/migrie/b/14316-atlas-debug-for-kids-who-dont-know-how-to-debug-and-want-to-do-other-things-good-too


📝 Commits (2)

  • d59a79a Don't just die if the user doesn't have the dx debugging tools
  • 6850679 redundant

📊 Changes

1 file changed (+40 additions, -4 deletions)

View changed files

📝 src/renderer/atlas/AtlasEngine.r.cpp (+40 -4)

📄 Description

This PR gives the atlas engine an attempt to retry a couple operations where it asks for debug flags when we're in debug mode. If you don't have the Graphics debugger and GPU profiler for DirectX installed, then these calls will fail, and we end up blowing up the renderer. Instead, just try again.

Originally, I actually thought I had hit #14082, but after sorting this out, it was just #14316.

closes #14316


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/microsoft/terminal/pull/15249 **Author:** [@zadjii-msft](https://github.com/zadjii-msft) **Created:** 4/27/2023 **Status:** ✅ Merged **Merged:** 4/27/2023 **Merged by:** [@DHowett](https://github.com/DHowett) **Base:** `main` ← **Head:** `dev/migrie/b/14316-atlas-debug-for-kids-who-dont-know-how-to-debug-and-want-to-do-other-things-good-too` --- ### 📝 Commits (2) - [`d59a79a`](https://github.com/microsoft/terminal/commit/d59a79a1dbb0379a1bbfad20c1bb8e2fefd9cb36) Don't just die if the user doesn't have the dx debugging tools - [`6850679`](https://github.com/microsoft/terminal/commit/68506793d80cf53ae99eccafbb4dfe1ff76c550e) redundant ### 📊 Changes **1 file changed** (+40 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `src/renderer/atlas/AtlasEngine.r.cpp` (+40 -4) </details> ### 📄 Description This PR gives the atlas engine an attempt to retry a couple operations where it asks for debug flags when we're in debug mode. If you don't have the Graphics debugger and GPU profiler for DirectX installed, then these calls will fail, and we end up blowing up the renderer. Instead, just try again. Originally, I actually thought I had hit #14082, but after sorting this out, it was just #14316. closes #14316 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-31 09:41:24 +00:00
Sign in to join this conversation.
No Label pull-request
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#30532