GNU bug report logs - #75495
[PATCH] ibuffer: New defcustom `ibuffer-human-readable-size'

Previous Next

Package: emacs;

Reported by: Daniel Mendler <mail <at> daniel-mendler.de>

Date: Sat, 11 Jan 2025 13:29:02 UTC

Severity: normal

Tags: patch

Done: Eli Zaretskii <eliz <at> gnu.org>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Robert Pluim <rpluim <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: Daniel Mendler <mail <at> daniel-mendler.de>, 75495 <at> debbugs.gnu.org
Subject: bug#75495: [PATCH] ibuffer: New defcustom `ibuffer-human-readable-size'
Date: Sun, 26 Jan 2025 12:16:16 +0100
>>>>> On Sat, 25 Jan 2025 10:23:39 +0200, Eli Zaretskii <eliz <at> gnu.org> said:

    >> From: Daniel Mendler <mail <at> daniel-mendler.de>
    >> Cc: Eli Zaretskii <eliz <at> gnu.org>
    >> Date: Sat, 25 Jan 2025 08:40:46 +0100
    >> 
    >> Pengji Zhang <me <at> pengjiz.com> writes:
    >> 
    >> > Daniel Mendler <mail <at> daniel-mendler.de> writes:
    >> >
    >> >> Yes, we should. I have attached the improved patch to this mail.
    >> >
    >> > It works great. Thanks!
    >> 
    >> Hello Eli,
    >> 
    >> does this patch (and my other Ibuffer patch) look okay to you, such that
    >> they can be installed? Thanks.

    Eli> Didn't yet have time to review it, will do soon.

It would be good if the patch consistently used either "size" or
"sizes" in the code and docstring, rather than a mix. Iʼd lean
slightly (but only slightly) to "sizes".

Robert
-- 




This bug report was last modified 119 days ago.

Previous Next


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