================
@@ -118,6 +118,6 @@ def test_launch_notifications(self):
# On Linux we get events for ld.so, [vdso], the binary and then all
libraries.
avg_solibs_added_per_event = round(
- float(total_solibs_added) / float(total_modules_added_events)
+ 10.0 * float(total_solibs_added) /
float(total_modules_added_events)
----------------
slydiman wrote:
> You can't really call it "number of libraries per event" anymore if you
> multiply by ten.
avg10_solibs_added_per_event ?
> Maybe you could just remove the round call, and compare to 1.0
No, because assertGreater() expects int parameters.
> Also, this test is very unhermetic
Right, and it seems that the behavior on Darwin is very different. I'd just
check `min_modules_per_event >= 1`.
https://github.com/llvm/llvm-project/pull/94672
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
