Skip to content

Commit f65e403

Browse files
committed
Move provider initialization-on-demand types to ContextProviders
1 parent 3417e48 commit f65e403

File tree

3 files changed

+16
-16
lines changed

3 files changed

+16
-16
lines changed

components/context/src/main/java/datadog/context/ContextBinder.java

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -24,11 +24,4 @@ public interface ContextBinder {
2424
static void register(ContextBinder binder) {
2525
ContextProviders.customBinder = binder;
2626
}
27-
28-
final class Provided {
29-
static final ContextBinder INSTANCE =
30-
null != ContextProviders.customBinder
31-
? ContextProviders.customBinder
32-
: new WeakMapContextBinder();
33-
}
3427
}

components/context/src/main/java/datadog/context/ContextManager.java

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -31,11 +31,4 @@ public interface ContextManager {
3131
static void register(ContextManager manager) {
3232
ContextProviders.customManager = manager;
3333
}
34-
35-
final class Provided {
36-
static final ContextManager INSTANCE =
37-
null != ContextProviders.customManager
38-
? ContextProviders.customManager
39-
: new ThreadLocalContextManager();
40-
}
4134
}

components/context/src/main/java/datadog/context/ContextProviders.java

Lines changed: 16 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,11 +6,25 @@ final class ContextProviders {
66
static volatile ContextManager customManager;
77
static volatile ContextBinder customBinder;
88

9+
private static final class ProvidedManager {
10+
static final ContextManager INSTANCE =
11+
null != ContextProviders.customManager
12+
? ContextProviders.customManager
13+
: new ThreadLocalContextManager();
14+
}
15+
16+
private static final class ProvidedBinder {
17+
static final ContextBinder INSTANCE =
18+
null != ContextProviders.customBinder
19+
? ContextProviders.customBinder
20+
: new WeakMapContextBinder();
21+
}
22+
923
static ContextManager manager() {
10-
return ContextManager.Provided.INSTANCE; // may be overridden by instrumentation
24+
return ProvidedManager.INSTANCE; // may be overridden by instrumentation
1125
}
1226

1327
static ContextBinder binder() {
14-
return ContextBinder.Provided.INSTANCE; // may be overridden by instrumentation
28+
return ProvidedBinder.INSTANCE; // may be overridden by instrumentation
1529
}
1630
}

0 commit comments

Comments
 (0)