bug#743: marked as done (23.0.60; linum-mode margin display wrong)

bug#743: marked as done (23.0.60; linum-mode margin display wrong)

[Emacs bug Tracking System]

[-- Attachment #1: Type: text/plain, Size: 848 bytes --]


Your message dated Tue, 26 Aug 2008 16:53:13 -0400
with message-id <87sksrqz0m.fsf@cyd.mit.edu>
and subject line Re: bug#743: 23.0.60; linum-mode margin display wrong
has caused the Emacs bug report #743,
regarding 23.0.60; linum-mode margin display wrong
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 don@donarmstrong.com
immediately.)


-- 
743: http://emacsbugs.donarmstrong.com/cgi-bin/bugreport.cgi?bug=743
Emacs Bug Tracking System
Contact don@donarmstrong.com with problems

[-- Attachment #2: Type: message/rfc822, Size: 2469 bytes --]

From: bojohan+mail@dd.chalmers.se (Johan Bockgård)
To: emacs-pretest-bug@gnu.org
Subject: 23.0.60; linum-mode margin display wrong
Date: Tue, 19 Aug 2008 21:29:39 +0200
Message-ID: <yoijd4k4x0po.fsf@remote5.student.chalmers.se>


$ emacs -Q -eval '(progn (linum-mode 1) (insert-char ?\n 100) (goto-line 12) (redisplay) (kill-line 10))'

The numbering in the margin of the first lines looks like:

  |
  |
  |
  |
  |
  |
  |
  |
  |
 1|
 1|
 1|
13|
14|
...

The first 12 lines have a display margin string that is 3 characters
long (space padded), but the margin is only 2 columns wide, so the last
digit is lost.

-- 
Johan Bockgård



[-- Attachment #3: Type: message/rfc822, Size: 1258 bytes --]

From: Chong Yidong <cyd@stupidchicken.com>
To: Markus Triska <markus.triska@gmx.at>
Cc: bojohan+mail@dd.chalmers.se (Johan Bockgård), 743-done@emacsbugs.donarmstrong.com
Subject: Re: bug#743: 23.0.60; linum-mode margin display wrong
Date: Tue, 26 Aug 2008 16:53:13 -0400
Message-ID: <87sksrqz0m.fsf@cyd.mit.edu>

> The following patch fixes this problem for me.

I've checked in the patch, together with Johan's suggestion of
inheriting the remaining face attributes of `linum' from the default
face.

Thanks.