bug#3971: marked as done (23.1.50; linum-mode kills right margin)

[Emacs bug Tracking System]

Your message dated Tue, 22 Sep 2009 02:09:43 +0200
with message-id <f7ccd24b0909211709l2c1f9093g589cc838e137b8f6@mail.gmail.com>
and subject line Re: bug#3971: 23.1.50; linum-mode kills right margin
has caused the Emacs bug report #3971,
regarding 23.1.50; linum-mode kills right margin
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@emacsbugs.donarmstrong.com
immediately.)


-- 
3971: http://emacsbugs.donarmstrong.com/cgi-bin/bugreport.cgi?bug=3971
Emacs Bug Tracking System
Contact owner@emacsbugs.donarmstrong.com with problems
> --- Begin Message ---
>
>
>
> Subject: 
>
> 23.1.50; linum-mode kills right margin
>
>
>
>
> Date: 
>
> Thu, 30 Jul 2009 06:38:26 +0200
>
>
>
>
> linum-mode kills the right margin. It should not touch it since it
> does not use it. The following patch fixes this:
>
> Index: linum.el
> ===================================================================
> RCS file: /sources/emacs/emacs/lisp/linum.el,v
> retrieving revision 1.11
> diff -c -b -r1.11 linum.el
> *** linum.el    18 Jan 2009 01:45:38 -0000      1.11
> --- linum.el    30 Jul 2009 04:35:10 -0000
> ***************
> *** 108,114 ****
>     (mapc #'delete-overlay linum-overlays)
>     (setq linum-overlays nil)
>     (dolist (w (get-buffer-window-list (current-buffer) nil t))
> !     (set-window-margins w 0)))
>
>   (defun linum-update-current ()
>     "Update line numbers for the current buffer."
> --- 108,114 ----
>     (mapc #'delete-overlay linum-overlays)
>     (setq linum-overlays nil)
>     (dolist (w (get-buffer-window-list (current-buffer) nil t))
> !     (set-window-margins w 0 (cdr (window-margins w)))))
>
>   (defun linum-update-current ()
>     "Update line numbers for the current buffer."
> ***************
> *** 163,169 ****
>               (overlay-put ov 'linum-str str))))
>         (forward-line)
>         (setq line (1+ line)))
> !     (set-window-margins win width)))
>
>   (defun linum-after-change (beg end len)
>     ;; update overlays on deletions, and after newlines are inserted
> --- 163,169 ----
>               (overlay-put ov 'linum-str str))))
>         (forward-line)
>         (setq line (1+ line)))
> !     (set-window-margins win width (cdr (window-margins win)))))
>
>   (defun linum-after-change (beg end len)
>     ;; update overlays on deletions, and after newlines are inserted
>
>
>
> In GNU Emacs 23.1.50.1 (i386-mingw-nt5.1.2600)
>  of 2009-06-30
> Windowing system distributor `Microsoft Corp.', version 5.1.2600
> configured using `configure --with-gcc (3.4) --no-opt --cflags
> -Ic:/g/include -fno-crossjumping'
>
>
> --- End Message ---
> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#3971: 23.1.50; linum-mode kills right margin
>
>
>
>
> Date: 
>
> Tue, 22 Sep 2009 02:09:43 +0200
>
>
>
>
> On Thu, Jul 30, 2009 at 07:24, Lennart Borgman
> <lennart.borgman@gmail.com> wrote:
>
> > Or, maybe it is better to redefine set-window-margins so that a nil
> > argument for a margin means "don't touch"...?
>
> I think that would be cleaner, because 0 is already available to set a
> margin to "no margin".
>
> But that's an interface change, so for now I've installed your change,
> which fixes the bug.
>
>     Juanma
>
> --- End Message ---