Skip to content

Commit b36a75e

Browse files
committed
Make autoPrefixes and instrumentEnable true by default
1 parent 069f2c5 commit b36a75e

File tree

2 files changed

+6
-4
lines changed

2 files changed

+6
-4
lines changed

CHANGELOG.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,8 @@
55
### New Features
66

77
### Changes
8+
9+
- Make autoPrefixes and instrumentEnable true by default
810
911
### Bugs fixed
1012

src/jvm/clojure/storm/Emitter.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ public class Emitter {
4242
static Keyword LINE_KEY = Keyword.intern(null, "line");
4343
static Keyword NS_KEY = Keyword.intern(null, "ns");
4444

45-
public static Var INSTRUMENTATION_ENABLE = Var.create(false).setDynamic();
45+
public static Var INSTRUMENTATION_ENABLE = Var.create(true).setDynamic();
4646

4747
private static ArrayList<String> instrumentationOnlyPrefixes = new ArrayList();
4848
private static ArrayList<String> instrumentationSkipPrefixes = new ArrayList();
@@ -63,8 +63,8 @@ public class Emitter {
6363
setInstrumentationEnable(Boolean.parseBoolean(instrumentationEnableProp));
6464

6565
String autoPrefixesProp = System.getProperty("clojure.storm.instrumentAutoPrefixes");
66-
if(autoPrefixesProp != null && Boolean.parseBoolean(autoPrefixesProp))
67-
{
66+
boolean autoPrefixes = autoPrefixesProp==null || Boolean.parseBoolean(autoPrefixesProp);
67+
if(autoPrefixes) {
6868
for (String autoPrefix : Utils.classpathSrcDirstRootNamespaces())
6969
{
7070
// This is kind of hacky because ClojureStorm shouldn't have anything related to
@@ -75,7 +75,7 @@ public class Emitter {
7575
}
7676

7777
}
78-
}
78+
}
7979

8080
for(String p : Utils.prefixesForPropStartingWith("clojure.storm.instrumentOnlyPrefixes"))
8181
addInstrumentationOnlyPrefix(p);

0 commit comments

Comments
 (0)