Use memoization to avoid costly operations. From: dcparker#82

BryanH · Nov 12, 2015 · 35f617f · 35f617f
1 parent f8c2651
commit 35f617f
Showing 1 changed file with 10 additions and 6 deletions.
diff --git a/lib/gmail/message.rb b/lib/gmail/message.rb
@@ -89,15 +89,19 @@ def archive!
 
     # Parsed MIME message object
     def message
-      require 'mail'
-      request,part = 'RFC822','RFC822'
-      request,part = 'BODY.PEEK[]','BODY[]' if @gmail.peek
-      _body = @gmail.in_mailbox(@mailbox) { @gmail.imap.uid_fetch(uid, request)[0].attr[part] }
-      @message ||= Mail.new(_body)
+      if @message
+        @message
+      else
+        require 'mail'
+        request,part = 'RFC822','RFC822'
+        request,part = 'BODY.PEEK[]','BODY[]' if @gmail.peek
+        _body = @gmail.in_mailbox(@mailbox) { @gmail.imap.uid_fetch(uid, request)[0].attr[part] }
+        @message = Mail.new(_body)
+      end
     end
 
     private
-    
+
     # Delegate all other methods to the Mail message
     def method_missing(*args, &block)
       if block_given?