[all-commits] [llvm/llvm-project] 4f2ccc: [lldb/Editline] Fix mistake in HistoryOperation ma...

Jonas Devlieghere via All-commits all-commits at lists.llvm.org
Fri Feb 14 14:07:35 PST 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 4f2cccc5ce89855bfc5cc44f9bb45fec745e7de9
      https://github.com/llvm/llvm-project/commit/4f2cccc5ce89855bfc5cc44f9bb45fec745e7de9
  Author: Jonas Devlieghere <jonas at devlieghere.com>
  Date:   2020-02-14 (Fri, 14 Feb 2020)

  Changed paths:
    M lldb/source/Host/common/Editline.cpp

  Log Message:
  -----------
  [lldb/Editline] Fix mistake in HistoryOperation mapping

In 0e9b0b6d11e882efec8505d97c4b65e1562e6715 I introduced the
HistoryOperation enum to navigate the history. While this fixed the
behavior of HistoryOperation::Older and HistoryOperation::Newer, it
confused the mapping for HistoryOperation::Oldest and
HistoryOperation::Newest.

I tried to write a PExpect test to make sure this doesn't regress, but
I'm unable to prime the history in such a way that it recalls a known
element. I suspect this is an LLDB bug, but the  most recent entry
doesn't get update with entries from the current session. I considered
spoofing the home directory but that needs to happen before libLLDB is
loaded and you'll need to account for the widechar support. If anyone
has another suggestion I'd love to hear it.




More information about the All-commits mailing list