diff options
author | Charles Cole <cc847m@att.com> | 2017-12-04 14:47:04 -0600 |
---|---|---|
committer | Charles Cole <cc847m@att.com> | 2017-12-04 21:09:13 +0000 |
commit | da75a5e8cd3e64fb8e7efa8db18e1a488870d0dc (patch) | |
tree | 158dda481852435ea14c12404815d510b21be444 /common-logging/src/main/java/org/onap/policy/common/logging/nsa | |
parent | e73c82943d4a1e6dffd7712837d50e58c01bef7b (diff) |
Fix sonar issues in ONAP-logging
Fixed the Sonar issues in ONAP-logging that required minimal refactoring
of the code. Many of the System.out and System.err issues I did not
fix, as I don't know what loggers to use.
Issue-ID: POLICY-458
Change-Id: I8eccb932b350a170b9cd680b90009e6612a0abcd
Signed-off-by: Charles Cole <cc847m@att.com>
Diffstat (limited to 'common-logging/src/main/java/org/onap/policy/common/logging/nsa')
2 files changed, 6 insertions, 5 deletions
diff --git a/common-logging/src/main/java/org/onap/policy/common/logging/nsa/LoggingContextFactory.java b/common-logging/src/main/java/org/onap/policy/common/logging/nsa/LoggingContextFactory.java index 78fffefe..b2ad76ca 100644 --- a/common-logging/src/main/java/org/onap/policy/common/logging/nsa/LoggingContextFactory.java +++ b/common-logging/src/main/java/org/onap/policy/common/logging/nsa/LoggingContextFactory.java @@ -32,6 +32,10 @@ public class LoggingContextFactory { public static class Builder { + + private LoggingContext fBase = null; + private boolean fShared = false; + public Builder withBaseContext ( LoggingContext lc ) { fBase = lc; @@ -48,8 +52,5 @@ public class LoggingContextFactory { return fShared ? new SharedContext ( fBase ) : new Slf4jLoggingContext ( fBase ); } - - private LoggingContext fBase = null; - private boolean fShared = false; } } diff --git a/common-logging/src/main/java/org/onap/policy/common/logging/nsa/impl/SharedContext.java b/common-logging/src/main/java/org/onap/policy/common/logging/nsa/impl/SharedContext.java index 7f4a1096..97f61699 100644 --- a/common-logging/src/main/java/org/onap/policy/common/logging/nsa/impl/SharedContext.java +++ b/common-logging/src/main/java/org/onap/policy/common/logging/nsa/impl/SharedContext.java @@ -32,6 +32,8 @@ import org.onap.policy.common.logging.nsa.SharedLoggingContext; */ public class SharedContext extends Slf4jLoggingContext implements SharedLoggingContext { + private final HashMap<String,String> fMap; + public SharedContext ( LoggingContext base ) { super ( base ); @@ -53,6 +55,4 @@ public class SharedContext extends Slf4jLoggingContext implements SharedLoggingC lc.put ( e.getKey(), e.getValue() ); } } - - private final HashMap<String,String> fMap; } |