From: Bertram Scharpf <software@bertram-scharpf.de>
Date: Wed, 27 Jun 2018 09:32:50 +0000 (+0200)
Subject: Only calculate mutt_fqdn when needed
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=a85e0a0356cbaad78713c8ceec9b52a521d223cf;p=neomutt

Only calculate mutt_fqdn when needed
---

diff --git a/send.c b/send.c
index b1c07fec8..2b01d0c44 100644
--- a/send.c
+++ b/send.c
@@ -1069,7 +1069,6 @@ static struct Address *set_reverse_name(struct Envelope *env)
 struct Address *mutt_default_from(void)
 {
   struct Address *addr = NULL;
-  const char *fqdn = mutt_fqdn(true);
 
   /* Note: We let $from override $realname here.
    *       Is this the right thing to do?
@@ -1077,17 +1076,19 @@ struct Address *mutt_default_from(void)
 
   if (From)
     addr = mutt_addr_copy(From);
-  else if (UseDomain)
-  {
-    addr = mutt_addr_new();
-    addr->mailbox =
-        mutt_mem_malloc(mutt_str_strlen(Username) + mutt_str_strlen(fqdn) + 2);
-    sprintf(addr->mailbox, "%s@%s", NONULL(Username), NONULL(fqdn));
-  }
-  else
-  {
+  else {
     addr = mutt_addr_new();
-    addr->mailbox = mutt_str_strdup(NONULL(Username));
+    if (UseDomain)
+    {
+      const char *fqdn = mutt_fqdn(true);
+      addr->mailbox =
+          mutt_mem_malloc(mutt_str_strlen(Username) + mutt_str_strlen(fqdn) + 2);
+      sprintf(addr->mailbox, "%s@%s", NONULL(Username), NONULL(fqdn));
+    }
+    else
+    {
+      addr->mailbox = mutt_str_strdup(NONULL(Username));
+    }
   }
 
   return addr;