Merge pull request #4177 from jimingham/type-it-right

Fix a mistyping introduced with the new container command.
apple · Sep 8, 2022 · 820bd00 · 820bd00
2 parents b1724cf + fc5e3b8
commit 820bd00
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/lldb/source/Interpreter/CommandInterpreter.cpp b/lldb/source/Interpreter/CommandInterpreter.cpp
@@ -1455,7 +1455,7 @@ void CommandInterpreter::GetHelp(CommandReturnObject &result,
     result.AppendMessage("Current user-defined container commands:");
     result.AppendMessage("");
     max_len = FindLongestCommandWord(m_user_mw_dict);
-    for (pos = m_user_dict.begin(); pos != m_user_mw_dict.end(); ++pos) {
+    for (pos = m_user_mw_dict.begin(); pos != m_user_mw_dict.end(); ++pos) {
       OutputFormattedHelpText(result.GetOutputStream(), pos->first, "--",
                               pos->second->GetHelp(), max_len);
     }

diff --git a/lldb/test/API/commands/command/container/TestContainerCommands.py b/lldb/test/API/commands/command/container/TestContainerCommands.py
@@ -69,7 +69,9 @@ def container_add(self):
         self.expect("test-multi test-multi-sub welcome friend", "Used the new command class",
                     substrs=["Hello friend, welcome again to LLDB"])
         self.expect("apropos welcome", "welcome should show up in apropos", substrs=["a docstring for the second Welcome"])
-
+        self.expect("help test-multi test-multi-sub welcome", "welcome should show up in help", substrs=["a docstring for the second Welcome"])
+        self.expect("help", "test-multi should show up in help", substrs=["test-multi"])
+
         # Now switch the default and make sure we can now delete w/o the overwrite option:
         self.runCmd("settings set interpreter.require-overwrite 0")
         self.runCmd("command script add -c welcome.WelcomeCommand test-multi test-multi-sub welcome")