From 28398dbdada82ba8335fa5d17953067c22fad2cc Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Andr=C3=A1s=20Veres-Szentkir=C3=A1lyi?= Date: Sat, 15 Feb 2014 21:33:08 +0100 Subject: [PATCH] removed unnecessary unboxing followed by reboxing Since id is a Long, Long.valueOf(long) unboxed the Long to a primitive long, then reboxed it into a Long instance, which was again unboxed to allow it to be set as an element of the array of longs. This commit reduces the number of boxings from 3 to 1. --- src/com/fsck/k9/fragment/MessageListFragment.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/com/fsck/k9/fragment/MessageListFragment.java b/src/com/fsck/k9/fragment/MessageListFragment.java index cacb19ea7..ff2f176f5 100644 --- a/src/com/fsck/k9/fragment/MessageListFragment.java +++ b/src/com/fsck/k9/fragment/MessageListFragment.java @@ -895,7 +895,7 @@ public class MessageListFragment extends SherlockFragment implements OnItemClick long[] selected = new long[mSelected.size()]; int i = 0; for (Long id : mSelected) { - selected[i++] = Long.valueOf(id); + selected[i++] = id; } outState.putLongArray(STATE_SELECTED_MESSAGES, selected); }