For the common case of "getMessage" where we actually have the object in

the list, use a less painful equality check than iteration.

This works because messageInfoHolders compare to each other using the
same key as message they contain.

src/com/fsck/k9/activity/MessageList.java

@@ -2396,7 +2396,19 @@ public class MessageList
         }
         public MessageInfoHolder getMessage(Message message)
         {
-            return getMessage(message.makeMessageReference());
+
+            synchronized (mAdapter.messages)
+            {
+                int index = mAdapter.messages.indexOf(message);
+                if (index == -1)
+                {
+                    return null;
+                }
+                else
+                {
+                    return mAdapter.messages.get(index);
+                }
+            }
         }
 
         // XXX TODO - make this not use a for loop