GNU bug report logs - #70902
[PATCH] Drop two invocations of 'apply'.

Previous Next

Package: mumi;

Reported by: Felix Lechner <felix.lechner <at> lease-up.com>

Date: Mon, 13 May 2024 04:31:02 UTC

Severity: normal

Tags: patch

Done: Arun Isaac <arunisaac <at> systemreboot.net>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Arun Isaac <arunisaac <at> systemreboot.net>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#70902: closed ([PATCH] Drop two invocations of 'apply'.)
Date: Thu, 13 Feb 2025 22:20:02 +0000
[Message part 1 (text/plain, inline)]
Your message dated Thu, 13 Feb 2025 22:19:20 +0000
with message-id <87jz9tzdw7.fsf <at> systemreboot.net>
and subject line Re: [PATCH] Drop two invocations of 'apply'.
has caused the debbugs.gnu.org bug report #70902,
regarding [PATCH] Drop two invocations of 'apply'.
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
70902: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=70902
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems
[Message part 2 (message/rfc822, inline)]
From: Felix Lechner <felix.lechner <at> lease-up.com>
To: bug-mumi <at> gnu.org
Cc: Felix Lechner <felix.lechner <at> lease-up.com>
Subject: [PATCH] Drop two invocations of 'apply'.
Date: Sun, 12 May 2024 21:20:12 -0700
They stood out among other nearby calls to render-html, and do not
appear to be needed.
---
 mumi/web/download.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/mumi/web/download.scm b/mumi/web/download.scm
index 5c12fca..d4eb09e 100644
--- a/mumi/web/download.scm
+++ b/mumi/web/download.scm
@@ -56,7 +56,7 @@ PATH."
      (html-response (email-headers entry)
                     (email-body entry)))
     (_
-     (apply render-html (unknown id)))))
+     (render-html (unknown id)))))
 
 (define (download-raw id msg-num)
   "Handle download of an attachment for bug ID, message number
@@ -77,4 +77,4 @@ PATH."
                 (content-disposition text/plain
                                      (filename . ,(format #f "~a-~a.mbox" id msg-num))))
               (list-ref emails msg-num))
-      (apply render-html (unknown id))))
+      (render-html (unknown id))))

base-commit: 99416ed5c7d950eaf54d52023a2efd975bccac92
-- 
2.41.0



[Message part 3 (message/rfc822, inline)]
From: Arun Isaac <arunisaac <at> systemreboot.net>
To: 70902-done <at> debbugs.gnu.org
Cc: Felix Lechner <felix.lechner <at> lease-up.com>
Subject: Re: [PATCH] Drop two invocations of 'apply'.
Date: Thu, 13 Feb 2025 22:19:20 +0000
I discovered these independently and fixed them in
0c9ed892e99189f74582a75f91fb8ab25e9fe498. Sorry about the duplicated
work. Thanks for reporting!


This bug report was last modified 154 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.