diff options
author | Abhishek Koneru <akoneru@redhat.com> | 2012-06-05 12:16:51 -0400 |
---|---|---|
committer | Endi Sukma Dewata <edewata@redhat.com> | 2012-06-05 14:46:08 -0500 |
commit | 4a012010233b0a5fd613c9d7734ee01be64e3834 (patch) | |
tree | 53affd70c3c2b5d0d909641f847b938e80e874c7 /base/common/src/com/netscape/certsrv/util/StatsEvent.java | |
parent | c2128339ba046ceb7ab32954ae07dbb4b41d4e73 (diff) | |
download | pki-4a012010233b0a5fd613c9d7734ee01be64e3834.tar.gz pki-4a012010233b0a5fd613c9d7734ee01be64e3834.tar.xz pki-4a012010233b0a5fd613c9d7734ee01be64e3834.zip |
Fixes for NULL_RETURN cases review comments
Diffstat (limited to 'base/common/src/com/netscape/certsrv/util/StatsEvent.java')
-rw-r--r-- | base/common/src/com/netscape/certsrv/util/StatsEvent.java | 6 |
1 files changed, 1 insertions, 5 deletions
diff --git a/base/common/src/com/netscape/certsrv/util/StatsEvent.java b/base/common/src/com/netscape/certsrv/util/StatsEvent.java index c431a3908..4137aa3b2 100644 --- a/base/common/src/com/netscape/certsrv/util/StatsEvent.java +++ b/base/common/src/com/netscape/certsrv/util/StatsEvent.java @@ -63,9 +63,7 @@ public class StatsEvent { */ public Enumeration<String> getSubEventNames() { Vector<String> names = new Vector<String>(); - for (String s : mSubEvents.keySet()) { - names.add(s); - } + names.addAll(mSubEvents.keySet()); return names.elements(); } @@ -83,9 +81,7 @@ public class StatsEvent { mTimeTaken = 0; mTimeTakenSqSum = 0; for (StatsEvent c : mSubEvents.values()) { - if (c != null) { c.resetCounters(); - } } } |