[Lldb-commits] [PATCH] D124785: [lldb/Core] Fix "sticky" long progress messages

Jonas Devlieghere via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed May 18 12:21:17 PDT 2022


JDevlieghere added inline comments.


================
Comment at: lldb/test/API/functionalities/progress_reporting/TestTrimmedProgressReporting.py:29-30
+        self.launch(use_colors=True)
+        # Set the terminal to a random width
+        term_width = random.randint(10, 42)
+        self.expect("set set show-progress true")
----------------
I understand the motivation, but I think it's a bad idea to intentionally introduce non-determinism in a test. Let's just make this a constant or repeat the test for two (fixed) widths.


================
Comment at: lldb/test/API/functionalities/progress_reporting/TestTrimmedProgressReporting.py:36-59
+        lines = []
+        with tempfile.NamedTemporaryFile() as tmpfile:
+            if self.TraceOn():
+                print("logfile: " + tmpfile.name)
+            with open(tmpfile.name, 'wb') as logfile:
+                self.child.logfile = logfile
+                self.expect("file " + self.getBuildArtifact("a.out"),
----------------
Why not use `self.child.expect_exact` with the exact sequence (including escape characters) that you're trying to match? This seems to reimplementing part of the pexpect functionality and I'm not sure we actually need it.


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

https://reviews.llvm.org/D124785



More information about the lldb-commits mailing list