[lldb] Make exe_ctx an optional argument in OptionValueProperties (NFC)

The majority of call sites are nullptr as the execution context.
Refactor OptionValueProperties to make the argument optional and
simplify all the callers.

1 files changed, 5 insertions, 4 deletions

diff --git a/lldb/unittests/Interpreter/TestOptionValue.cpp b/lldb/unittests/Interpreter/TestOptionValue.cpp
index 30687d6a6ede..64adcff243fd 100644
--- a/lldb/unittests/Interpreter/TestOptionValue.cpp
+++ b/lldb/unittests/Interpreter/TestOptionValue.cpp
@@ -16,6 +16,7 @@ class Callback {
 public:
   virtual void Invoke() const {}
   void operator()() const { Invoke(); }
+
 protected:
   ~Callback() = default;
 };
@@ -101,12 +102,11 @@ public:
   }
 
   OptionValueDictionary *GetDictionary() {
-    return GetPropertyAtIndexAsOptionValueDictionary(nullptr, m_dict_index);
+    return GetPropertyAtIndexAsOptionValueDictionary(m_dict_index);
   }
 
   OptionValueFileSpecList *GetFileList() {
-    return GetPropertyAtIndexAsOptionValueFileSpecList(nullptr,
-                                                       m_file_list_index);
+    return GetPropertyAtIndexAsOptionValueFileSpecList(m_file_list_index);
   }
 
 private:
@@ -170,5 +170,6 @@ TEST(TestProperties, DeepCopy) {
   dict_copy_ptr->SetValueFromString("C=3", eVarSetOperationAppend);
 
   // Trigger the callback second time.
-  file_list_copy_ptr->SetValueFromString("0 another/path", eVarSetOperationReplace);
+  file_list_copy_ptr->SetValueFromString("0 another/path",
+                                         eVarSetOperationReplace);
 }