change shortcut for 'step out' from 'o' to 'f'

[llunak]

Summary

This makes it consistent with gdb tui, where 'f' is 'finish'.
See the discussion at https://reviews.llvm.org/D68541 .

[kwk]

Are there no tests for this code?

[llunak]

In D68909#1736931, @kwk wrote:

Are there no tests for this code?

https://reviews.llvm.org/D68541 adds one that also covers this.

[llunak]

Updated for current git.

lldb/source/Core/IOHandlerCursesGUI.cpp

static curses::KeyHelp g_source_view_key_help[] = {
         {KEY_PPAGE, "Page up"},
         {KEY_NPAGE, "Page down"},
         {'b', "Set breakpoint on selected source/disassembly line"},
         {'c', "Continue process"},
         {'D', "Detach with process suspended"},
         {'h', "Show help dialog"},
         {'n', "Step over (source line)"},
         {'N', "Step over (single instruction)"},
-        {'o', "Step out"},
+        {'f', "Step out (finish)"},
         {'s', "Step in (source line)"},
         {'S', "Step in (single instruction)"},
         {',', "Page up"},
         {'.', "Page down"},
         {'\0', nullptr}};
     return g_source_view_key_help;
   }
 
case 'c':
       {
         ExecutionContext exe_ctx =
             m_debugger.GetCommandInterpreter().GetExecutionContext();
         if (exe_ctx.HasProcessScope())
           exe_ctx.GetProcessRef().Resume();
       }
       return eKeyHandled;
 
-    case 'o':
-      // 'o' == step out
+    case 'f':
+      // 'f' == step out (finish)
       {
         ExecutionContext exe_ctx =
             m_debugger.GetCommandInterpreter().GetExecutionContext();
         if (exe_ctx.HasThreadScope() &&
             StateIsStoppedState(exe_ctx.GetProcessRef().GetState(), true)) {
           exe_ctx.GetThreadRef().StepOut();
         }
       }