JDevlieghere wrote: > @walter-erquinigo > > > Please write a section in the lldb-dap documentation on how to integrate > > with other debugger extensions via these commands? > > That was a great suggestion. Added text to near the end of the `README.md`. > Please feel free to LMK if you have different wording/framing in mind. Happy > to incorporate.
Given that we use the README in the Marketplace, maybe https://lldb.llvm.org/resources/lldbdap.html would be a more appropriate place for this? It's probably not relevant to 99% of the users of the extension. > > @JDevlieghere > > > Are you working on an extension that will take advantage of this, or are > > the commands added in this PR purely speculative? > > We have an internal debugger extension (our main debugging entry point), and > we are actively working on leveraging the `lldb-dap` extension's features > (e.g. Server Mode, and other settings). > > In long term, we think the most correct way would be to leverage this > open-source extension as much as possible and upstream features wherever > makes sense. > > > it'll be hard to figure out in the future if we can change these or have to > > treat them like ABI. > > Yes, that's a tradeoff. Hence my approach of having these commands at the > highest level possible (the VS Code API level, which hopefully won't be > changed by Microsoft), so as to reduce the chance of having to change them > later. > > WDYT? Ack, thanks for the context, that all sounds reasonable. https://github.com/llvm/llvm-project/pull/151827 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
