opendev/gerrit
Code Issues Proposed changes

Merge "Use valueOf instead of constructor for better time/space perf"

Browse Source
This commit is contained in:
ekempin
2016-09-21 07:22:06 +00:00
committed by Gerrit Code Review
parent 9ec0df0d98 a28f4ec61e
commit f01ae3806e
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
gerrit-server/src/main/java/com/google/gerrit/server/extensions/events/EventUtil.java
View File

@@ -94,7 +94,7 @@ public class EventUtil {
Map<String, Short> approvals, Timestamp ts) {
Map<String, ApprovalInfo> result = new HashMap<>();
for (Map.Entry<String, Short> e : approvals.entrySet()) {
Integer value = e.getValue() != null ? new Integer(e.getValue()) : null;
Integer value = e.getValue() != null ? Integer.valueOf(e.getValue()) : null;
result.put(e.getKey(),
ChangeJson.getApprovalInfo(a.getId(), value, null, ts));
}