Remove unnecessary strcmp for mutt_view_attachment().

mutt_rfc1524_expand_filename() runs the result through
mutt_adv_mktemp(), which will sanitize and relocate the filename under
$tmpdir.  The strcmp() is unneeded and distracting to the program
logic; none of the other routines perform or need this check.

1 files changed, 1 insertions, 1 deletions

diff --git a/attach.c b/attach.c
index 04a5610b..552a1767 100644
--- a/attach.c
+++ b/attach.c
@@ -415,7 +415,7 @@ int mutt_view_attachment (FILE *fp, BODY *a, int flag, HEADER *hdr,
     if (mutt_rfc1524_expand_filename (entry->nametemplate, fname,
                                       tempfile))
     {
-      if (fp == NULL && mutt_strcmp(mutt_b2s (tempfile), a->filename))
+      if (fp == NULL)
       {
 	/* send case: the file is already there */
 	if (safe_symlink (a->filename, mutt_b2s (tempfile)) == -1)