Avoid spurious breakpoint-setting failure in DAP
authorTom Tromey <tromey@adacore.com>
Tue, 5 Sep 2023 12:55:54 +0000 (06:55 -0600)
committerTom Tromey <tromey@adacore.com>
Tue, 12 Sep 2023 16:31:26 +0000 (10:31 -0600)
commitf8ab027008863bd06513053ea9f84e1116c1cf73
tree628e1cb3b83a5d9c9e0b5b3165fff319be75e768
parentd1722abe60ca7c330210aa97c8ec52ff98644206
Avoid spurious breakpoint-setting failure in DAP

A user pointed out that if a DAP setBreakpoints request has a 'source'
field in a SourceBreakpoint object, then the gdb DAP implementation
will throw an exception.

While SourceBreakpoint does not allow 'source' in the spec, it seems
better to me to accept it.  I don't think we should fully go down the
"Postel's Law" path -- after all, we have the type-checker -- but at
the same time, if we do send errors, they should be intentional and
not artifacts of the implementation.

Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=30820
gdb/python/lib/gdb/dap/breakpoint.py
gdb/testsuite/gdb.dap/basic-dap.exp