[all-commits] [llvm/llvm-project] 075de2: Save and restore previous terminal after setting t...

Pavel Labath via All-commits all-commits at lists.llvm.org
Thu Mar 11 01:47:27 PST 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 075de2d8a7567a6a39e8477407960aa2545b68c2
      https://github.com/llvm/llvm-project/commit/075de2d8a7567a6a39e8477407960aa2545b68c2
  Author: Augusto Noronha <augusto2112 at me.com>
  Date:   2021-03-11 (Thu, 11 Mar 2021)

  Changed paths:
    M llvm/lib/Support/Unix/Process.inc

  Log Message:
  -----------
  Save and restore previous terminal after setting the terminal for checking if terminal supports colors.

The call to "set_curterm" inside the "terminalHasColors" function breaks
the EditLine configuration on some Linux distributions, causing certain
characters that have functions bound to them to not show up and
backspace to stop deleting characters (only visually). This patch
ensures that term struct is restored after the routine for cheking if
terminal supports colors is done, which fixes the aforementioned issue.

Reviewed By: labath

Differential Revision: https://reviews.llvm.org/D95230


  Commit: 3d47f1f9b826003df4c19767d433095df89d1bc5
      https://github.com/llvm/llvm-project/commit/3d47f1f9b826003df4c19767d433095df89d1bc5
  Author: Pavel Labath <pavel at labath.sk>
  Date:   2021-03-11 (Thu, 11 Mar 2021)

  Changed paths:
    R lldb/test/Shell/Breakpoint/implicit_const_form_support.test

  Log Message:
  -----------
  [lldb] Remove implicit_const_form_support.test

It is superseded by dwarf5-implicit-const.s (added in D98197), which tests it more thoroughly.


Compare: https://github.com/llvm/llvm-project/compare/bafe418d12e9...3d47f1f9b826


More information about the All-commits mailing list