guix-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] emacs: Fill package description in Info buffers.


From: Ludovic Courtès
Subject: Re: [PATCH] emacs: Fill package description in Info buffers.
Date: Tue, 14 Jul 2015 15:15:08 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux)

Alex Kost <address@hidden> skribis:

> Since there is a buffer-local 'fill-column' variable, I think we may use
> it instead of hard-coding '72', so that a user could change it, for
> example like this:
>
> (add-hook 'guix-package-info-mode-hook
>           (lambda () (setq fill-column 60)))

Indeed, it’s better this way.

>> +                (guix-format-insert val (and (facep face) face))
>> +                (fill-region begin (point))
>
> I think it's better to use 'guix-get-filled-string' instead of
> (let ((begin (point))) ... (fill-region begin (point)))

Agreed, I hadn’t noticed that one.

> Actually, I think that preserving the original formatting used in
> "description" fields is a feature, not a bug, as it allows to find too
> long lines or other related things and then to fix those.
>
> (For example, I've just accidentally found that the first line of
> "pulseaudio" description is too long, and when I looked at the source, I
> also noticed that the description ends with a space.  With your change
> it wouldn't be possible to notice :-))

I think I would turn to the source file when I want to nitpick on
description formatting.  ;-)

> But as we can always add another option for a user to choose the variant
> he wants, I agree.  What about the attached modified patch?
>
> Also as it may concern synopsis (when a current window is too small to
> fit the synopsis), I renamed the patch into "emacs: Fill package heading
> in Info buffers."

Your patch looks better indeed.  Please push, thank you!

Ludo’.



reply via email to

[Prev in Thread] Current Thread [Next in Thread]