From 2b3d7ca783a3011eef4f8eda476798943ff0ff29 Mon Sep 17 00:00:00 2001 From: peter Date: Wed, 8 Mar 2017 22:02:01 +0100 Subject: [PATCH] less spam when initializing performance test timings, don't throw --- .../com/intellij/testFramework/CpuTimings.java | 15 +++++++++++---- 1 file changed, 11 insertions(+), 4 deletions(-) diff --git a/platform/testFramework/src/com/intellij/testFramework/CpuTimings.java b/platform/testFramework/src/com/intellij/testFramework/CpuTimings.java index f4ae11a6709e..0657d629b812 100644 --- a/platform/testFramework/src/com/intellij/testFramework/CpuTimings.java +++ b/platform/testFramework/src/com/intellij/testFramework/CpuTimings.java @@ -50,16 +50,23 @@ class CpuTimings { return "CpuTimings{" + average + ", raw=" + Arrays.toString(rawData) + ", sd=" + myStandardDeviation + '}'; } + @SuppressWarnings("UseOfSystemOutOrSystemErr") static CpuTimings calcStableCpuTiming() { - for (int i = 0; i < 200; i++) { + for (int i = 0;; i++) { CpuTimings timings = calcCpuTiming(); if (timings.myStandardDeviation < 1.8) { return timings; } - //noinspection UseOfSystemOutOrSystemErr - System.out.println("Unstable timings: " + timings); + if (i == 100) { + System.out.println("Cannot calculate timings that are stable enough, giving up"); + return timings; + } + if (i > 3) { + System.out.println(i + ": Unstable timings: " + timings); + } + System.gc(); } - throw new IllegalStateException("Cannot calculate timings that are stable enough"); + } static CpuTimings calcCpuTiming() {