[Lldb-commits] [lldb] [lldb][lldb-dap] use the new protocol for setVariable requests. (PR #137803)

John Harrison via lldb-commits lldb-commits at lists.llvm.org
Tue Apr 29 14:03:27 PDT 2025


================
@@ -430,17 +430,20 @@ class ScopesRequestHandler : public LegacyRequestHandler {
   void operator()(const llvm::json::Object &request) const override;
 };
 
-class SetVariableRequestHandler : public LegacyRequestHandler {
+class SetVariableRequestHandler final
----------------
ashgti wrote:

Probably my lack of c++ knowledge, but what impact does changing this to `final` have? Should we update all our handlers to be final?

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


More information about the lldb-commits mailing list