labath accepted this revision.
labath added a comment.
This revision is now accepted and ready to land.

Thanks for getting to the bottom of this. Adding the \n check is fine. The 
"expect" function was meant to be an pexpect equivalent of the "expect" 
function in regular tests (which also just handles a single command). If anyone 
is doing something fancier than that, he should probably interact with pexpect 
directly.



================
Comment at: lldb/packages/Python/lldbsuite/test/terminal/TestEditline.py:29-33
         3. Send "c"         -> "el c[]ommand"
         4. Ctrl+left twice  -> "[]el command"
         5. Send "h"         -> "h[]el command"
         6. Ctrl+right       -> "hel[] command"
         7. Send "p"         -> "help command"
----------------
Maybe update the example to use the command "print" too?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D70324/new/

https://reviews.llvm.org/D70324



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to