diff options
author | Jim Hahn <jrh3@att.com> | 2017-11-29 17:01:15 -0500 |
---|---|---|
committer | Jim Hahn <jrh3@att.com> | 2017-12-01 15:30:53 -0500 |
commit | 0fe7bd5eb21ca5a8daef531ade9156bb5c2e0207 (patch) | |
tree | e98a0f230e86be54e123a7c555e52a5349d01e7a /feature-healthcheck | |
parent | 49e60f1b0c95f63ce48fde526cc5c1ec1f046d74 (diff) |
Modified state mgmt to fix some sonar issues
Reordered modifiers.
Reordered variables, methods, and constructors.
Removed useless parentheses.
Removed unneeded "catch" clauses.
Extracted nested try blocks into their own method.
Replaced a string with a constant.
Removed extra thrown exceptions when they are unnecessary (i.e., they're
subclasses of RuntimeException, or the method is already declared to
throw an Exception).
Replaced a large anonymous class with a named, nested class.
Separated variable declarations onto individual lines.
Changed "String args[]" to "String[] args".
Replaced if-then-else by single return statement.
Invoked super() inside empty, default constructor.
Removed Thread.sleep() calls from junit test per comments on 11/29.
Commented out Thread.sleep() in junit tests, as they don't appear to
be necessary. If that turns out to be untrue, then CountdownLatch.await()
can be used instead.
Sonar complained about useless assignments to "phase", but those did
not appear to be useless.
Did not remove commented-out lines, as they may be needed when debugging.
Change-Id: I90ba6f7317a18a10ce1b881cfc6d21a602171ff5
Issue-ID: POLICY-469
Signed-off-by: Jim Hahn <jrh3@att.com>
Diffstat (limited to 'feature-healthcheck')
0 files changed, 0 insertions, 0 deletions