From cb32bc97dd54f09ea78104a38b0339aee01daaea Mon Sep 17 00:00:00 2001 From: Nikolay Krasko Date: Mon, 19 Sep 2016 18:12:59 +0300 Subject: [PATCH] Remove logging in LockBasedStorageManager creation as it all the same useless HEAD - 3 leads to "org.jetbrains.kotlin.storage.LockBasedStorageManager.createWithExceptionHandling(LockBasedStorageManager.java:65)" debug string --- .../kotlin/storage/LockBasedStorageManager.java | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/core/util.runtime/src/org/jetbrains/kotlin/storage/LockBasedStorageManager.java b/core/util.runtime/src/org/jetbrains/kotlin/storage/LockBasedStorageManager.java index 22626294197..78f4abaa135 100644 --- a/core/util.runtime/src/org/jetbrains/kotlin/storage/LockBasedStorageManager.java +++ b/core/util.runtime/src/org/jetbrains/kotlin/storage/LockBasedStorageManager.java @@ -80,18 +80,15 @@ public class LockBasedStorageManager implements StorageManager { } public LockBasedStorageManager() { - this(getPointOfConstruction(), ExceptionHandlingStrategy.THROW, new ReentrantLock()); + this(defaultDebugName(), ExceptionHandlingStrategy.THROW, new ReentrantLock()); } protected LockBasedStorageManager(@NotNull ExceptionHandlingStrategy exceptionHandlingStrategy) { - this(getPointOfConstruction(), exceptionHandlingStrategy, new ReentrantLock()); + this(defaultDebugName(), exceptionHandlingStrategy, new ReentrantLock()); } - private static String getPointOfConstruction() { - StackTraceElement[] trace = Thread.currentThread().getStackTrace(); - // we need to skip frames for getStackTrace(), this method and the constructor that's calling it - if (trace.length <= 3) return ""; - return trace[3].toString(); + private static String defaultDebugName() { + return ""; } @Override @@ -478,7 +475,7 @@ public class LockBasedStorageManager implements StorageManager { @NotNull LockBasedStorageManager base, @NotNull ExceptionHandlingStrategy newStrategy ) { - return new LockBasedStorageManager(getPointOfConstruction(), newStrategy, base.lock); + return new LockBasedStorageManager(defaultDebugName(), newStrategy, base.lock); } @NotNull