[Lldb-commits] [lldb] [lldb-dap] Use structured types for stepInTargets request (PR #142439)

[John Harrison via lldb-commits]

================
@@ -414,6 +415,34 @@ bool fromJSON(const llvm::json::Value &, SteppingGranularity &,
               llvm::json::Path);
 llvm::json::Value toJSON(const SteppingGranularity &);
 
+/// A `StepInTarget` can be used in the `stepIn` request and determines into
+/// which single target the `stepIn` request should step.
+struct StepInTarget {
+  /// Unique identifier for a step-in target.
+  lldb::addr_t id = LLDB_INVALID_ADDRESS;
+
+  /// The name of the step-in target (shown in the UI).
+  std::string label;
+
+  /// The line of the step-in target.
+  std::optional<uint32_t> line = LLDB_INVALID_LINE_NUMBER;
----------------
ashgti wrote:

Since this has a default, you can drop the `std::optional`.

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