Skip to content

Add support for HitCount and HitCondition#1548

Merged
WardenGnaw merged 5 commits into
mainfrom
dev/waan/hit-condition-validation
May 11, 2026
Merged

Add support for HitCount and HitCondition#1548
WardenGnaw merged 5 commits into
mainfrom
dev/waan/hit-condition-validation

Conversation

@WardenGnaw
Copy link
Copy Markdown
Member

This PR adds in showing the Hit count for a breakpoint and for setting a HitCondition.

Enables m_supportsHitConditionalBreakpoints, OpenDebugAD7 now properly builds a AD7BreakpointRequest with a hitCondition, AD7BoundBreakpoint now properly tracks hits and will prevent a break event if theres a condition set.

Added CppTests and updated DebuggerTesting for associated commands.

Addresses: #472

Comment thread src/MIDebugEngine/AD7.Impl/AD7BoundBreakpoint.cs
@WardenGnaw WardenGnaw force-pushed the dev/waan/hit-condition-validation branch 2 times, most recently from e32937b to f573e17 Compare March 14, 2026 22:36
Comment thread src/MIDebugEngine/AD7.Impl/AD7BoundBreakpoint.cs
Comment thread src/MIDebugEngine/Engine.Impl/BreakpointManager.cs
Comment thread src/MIDebugEngine/AD7.Impl/AD7PendingBreakpoint.cs
Comment thread src/MIDebugEngine/AD7.Impl/AD7PendingBreakpoint.cs Outdated
@WardenGnaw WardenGnaw force-pushed the dev/waan/hit-condition-validation branch from f573e17 to e61f18d Compare March 20, 2026 23:09
Comment thread src/MIDebugEngine/Engine.Impl/BreakpointManager.cs Outdated
WardenGnaw added 5 commits May 7, 2026 17:53
This PR adds in showing the Hit count for a breakpoint and for setting a
HitCondition.

Enables m_supportsHitConditionalBreakpoints, OpenDebugAD7 now properly
builds a AD7BreakpointRequest with a hitCondition, AD7BoundBreakpoint
now properly tracks hits and will prevent a break event if theres a
condition set.

Added CppTests and updated DebuggerTesting for associated commands.

Addresses: #472
@WardenGnaw WardenGnaw force-pushed the dev/waan/hit-condition-validation branch from 47f59f5 to a6f0d75 Compare May 8, 2026 00:54
@WardenGnaw
Copy link
Copy Markdown
Member Author

@gregg-miskelly When you have the time, could you review the latest changes?

@WardenGnaw WardenGnaw merged commit 93d5334 into main May 11, 2026
6 checks passed
@WardenGnaw WardenGnaw deleted the dev/waan/hit-condition-validation branch May 11, 2026 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants