diff --git a/lib/mail/message.rb b/lib/mail/message.rb
index <HASH>..<HASH> 100644
--- a/lib/mail/message.rb
+++ b/lib/mail/message.rb
@@ -1939,7 +1939,6 @@ module Mail
 
     def add_required_fields
       add_multipart_mixed_header    unless !body.multipart?
-      body = nil                    if body.nil?
       add_message_id                unless (has_message_id? || self.class == Mail::Part)
       add_date                      unless has_date?
       add_mime_version              unless has_mime_version?
Remove confusing no-op

This line previously always set the (unused) local variable body to nil.

I assume it meant to read `self.body = nil if body.nil?` however
`body.nil?` cannot happen as the previous line would crash.

Closer inspection reveals that the initialize method calls either
init_with_string or init_with_hash, both of which eventually set the
body to non-nil.