[Lldb-commits] [lldb] [lldb/crashlog] Make interactive mode the new default (PR #94575)

[Alex Langford via lldb-commits]

================
@@ -1794,8 +1803,36 @@ def SymbolicateCrashLogs(debugger, command_args, result, is_command):
         result.SetError(str(e))
         return
 
+    # To avoid breaking existing users, we should keep supporting legacy flags
+    # even if we don't use them / advertise them anymore.
+    if options.interactive:
+        options.mode = CrashLogLoadingMode.interactive
+    elif options.batch:
+        options.mode = CrashLogLoadingMode.batch
----------------
bulbazord wrote:

Might be a good place to check if `options.mode` was set before `options.interactive` and/or `options.batch`? What happens if I do something like `-m interactive -i -b`?

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