From 77434168a4c9095eed1b54fd88ea4d1cd3891b40 Mon Sep 17 00:00:00 2001 From: Connor Peet Date: Tue, 12 Nov 2024 10:19:52 -0800 Subject: [PATCH] fix: clarify `ContinuedEvent.allThreadsContinued` (#514) Fixes #513 --- changelog.md | 1 + debugAdapterProtocol.json | 4 ++-- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/changelog.md b/changelog.md index c469c89..4d7884c 100644 --- a/changelog.md +++ b/changelog.md @@ -8,6 +8,7 @@ sectionid: changelog * 1.70.x * Clarify how `StackTraceArguments.format` applies + * Clarify the default behavior of `ContinuedEvent.allThreadsContinued` * 1.69.x * Clarify the flow diagram to start a debug session diff --git a/debugAdapterProtocol.json b/debugAdapterProtocol.json index 9e148d6..7c7fea1 100644 --- a/debugAdapterProtocol.json +++ b/debugAdapterProtocol.json @@ -249,7 +249,7 @@ }, "allThreadsContinued": { "type": "boolean", - "description": "If `allThreadsContinued` is true, a debug adapter can announce that all threads have continued." + "description": "If omitted or set to `true`, this event signals to the client that all threads have been resumed. The value `false` indicates that not all threads were resumed." } }, "required": [ "threadId" ] @@ -1673,7 +1673,7 @@ "properties": { "allThreadsContinued": { "type": "boolean", - "description": "The value true (or a missing property) signals to the client that all threads have been resumed. The value false indicates that not all threads were resumed." + "description": "If omitted or set to `true`, this response signals to the client that all threads have been resumed. The value `false` indicates that not all threads were resumed." } } }