[Lldb-commits] [lldb] [lldb-dap] Add feature to remember last non-empty expression. (PR #107485)

Walter Erquinigo via lldb-commits lldb-commits at lists.llvm.org
Sat Sep 14 10:25:17 PDT 2024


================
@@ -1364,8 +1364,14 @@ void request_evaluate(const llvm::json::Object &request) {
   std::string expression = GetString(arguments, "expression").str();
   llvm::StringRef context = GetString(arguments, "context");
 
-  if (context == "repl" && g_dap.DetectExpressionContext(frame, expression) ==
-                               ExpressionContext::Command) {
+  if (context == "repl" &&
+      (expression.empty() ?
+       g_dap.last_nonempty_var_expression.empty() :
+       g_dap.DetectExpressionContext(frame, expression) ==
----------------
walter-erquinigo wrote:

it's hard to read this. Please move this ternary check into another variable with a good name.

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


More information about the lldb-commits mailing list