Mercurial > hg > truffle
diff graal/com.oracle.graal.debug/src/com/oracle/graal/debug/internal/DebugScope.java @ 13551:ba72961e0d41
removed ability for the log indentation mechanism to also enable/disable logging which should be controlled only by the -G:Log command line option
author | Doug Simon <doug.simon@oracle.com> |
---|---|
date | Wed, 08 Jan 2014 15:40:06 +0100 |
parents | 8ea968b6dba9 |
children | e34f406850e5 |
line wrap: on
line diff
--- a/graal/com.oracle.graal.debug/src/com/oracle/graal/debug/internal/DebugScope.java Wed Jan 08 14:50:05 2014 +0100 +++ b/graal/com.oracle.graal.debug/src/com/oracle/graal/debug/internal/DebugScope.java Wed Jan 08 15:40:06 2014 +0100 @@ -35,13 +35,11 @@ private static final String INDENTATION_INCREMENT = " "; final String indent; - boolean enabled; final IndentImpl parentIndent; - IndentImpl(IndentImpl parentIndent, boolean enabled) { + IndentImpl(IndentImpl parentIndent) { this.parentIndent = parentIndent; this.indent = (parentIndent == null ? "" : parentIndent.indent + INDENTATION_INCREMENT); - this.enabled = enabled; } private void printScopeName() { @@ -56,7 +54,7 @@ @Override public void log(String msg, Object... args) { - if (enabled) { + if (isLogEnabled()) { printScopeName(); output.println(indent + String.format(msg, args)); lastUsedIndent = this; @@ -64,13 +62,8 @@ } @Override - public void setEnabled(boolean enabled) { - this.enabled = enabled; - } - - @Override public Indent indent() { - lastUsedIndent = new IndentImpl(this, enabled); + lastUsedIndent = new IndentImpl(this); return lastUsedIndent; } @@ -115,6 +108,7 @@ private boolean meterEnabled; private boolean timeEnabled; private boolean dumpEnabled; + private boolean logEnabled; private PrintStream output; @@ -141,10 +135,10 @@ this.context = context; this.qualifiedName = qualifiedName; if (parent != null) { - lastUsedIndent = new IndentImpl(parent.lastUsedIndent, parent.isLogEnabled()); + lastUsedIndent = new IndentImpl(parent.lastUsedIndent); logScopeName = !parent.qualifiedName.equals(qualifiedName); } else { - lastUsedIndent = new IndentImpl(null, false); + lastUsedIndent = new IndentImpl(null); logScopeName = true; } @@ -178,11 +172,7 @@ } public boolean isLogEnabled() { - return lastUsedIndent.enabled; - } - - public void setLogEnabled(boolean enabled) { - lastUsedIndent.setEnabled(enabled); + return logEnabled; } public boolean isMeterEnabled() { @@ -251,7 +241,6 @@ newScope = this.createChild(name, context); } instanceTL.set(newScope); - newScope.setLogEnabled(this.isLogEnabled()); newScope.updateFlags(); return newScope; } @@ -297,10 +286,8 @@ meterEnabled = config.isMeterEnabled(); timeEnabled = config.isTimeEnabled(); dumpEnabled = config.isDumpEnabled(); + logEnabled = config.isLogEnabled(); output = config.output(); - if (config.isLogEnabled()) { - setLogEnabled(true); - } } }