Code Review / policy / engine.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: bdb9bf5) | patch
Remove technical debt 91/30291/1
authorPamela Dragosh <pdragosh@research.att.com>
Mon, 5 Feb 2018 14:38:29 +0000 (09:38 -0500)
committerPamela Dragosh <pdragosh@research.att.com>
Mon, 5 Feb 2018 14:40:12 +0000 (09:40 -0500)
commit153ec0fda79b07ca1ecf7016f1e7f4e2c792f04e
tree2913356137f72f97da8feab10d811e3d8ef2145ctree | snapshot
parentbdb9bf58d7a80f43c3ce558e6fe59c3bc0f6f8bdcommit | diff
Remove technical debt

* String literals on the left
* Unnecessary initialization of variables
* Change ArrayList to using List
* Remove extra semicolons
* Empty Constructor comments
* Useless parenthesis

I also added a comment on some code that is suspicious. Not comfortable
with changing that code.

Issue-ID: POLICY-482
Change-Id: I89d889737d398d047fab4b25cb5d962ee1ecdd03
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>
POLICY-SDK-APP/src/main/java/org/onap/policy/admin/PolicyManagerServlet.java diff | blob | history
POLICY-SDK-APP/src/main/java/org/onap/policy/controller/AutoPushController.java diff | blob | history
POLICY-SDK-APP/src/main/java/org/onap/policy/controller/CreateClosedLoopFaultController.java diff | blob | history
POLICY-SDK-APP/src/main/java/org/onap/policy/controller/CreateFirewallController.java diff | blob | history
-- Archived 06/09/2021