[Lldb-commits] [lldb] [lldb-dap] Remove an incorrect assumption on reverse requests. (PR #136210)

[Jonas Devlieghere via lldb-commits]

JDevlieghere wrote:

Yeah, according to the [spec](https://microsoft.github.io/debug-adapter-protocol/specification#Base_Protocol_ProtocolMessage) all protocol messages have a sequence number and the sequence number is 1-based:

> Sequence number of the message (also known as message ID). The `seq` for
> the first message sent by a client or debug adapter is 1, and for each
> subsequent message is 1 greater than the previous message sent by that
> actor. `seq` can be used to order requests, responses, and events, and to
> associate requests with their corresponding responses. For protocol
> messages of type `request` the sequence number can be used to cancel the
> request.

The spec says "can" but the sequence number is not optional. 

https://github.com/llvm/llvm-project/pull/136210