unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Eli Zaretskii <eliz@gnu.org>
To: David Reitter <david.reitter@gmail.com>
Cc: mituharu@math.s.chiba-u.ac.jp, emacs-devel@gnu.org
Subject: Re: Truncating scroll runs that copy to where we copied to
Date: Tue, 22 Nov 2011 01:04:45 -0500	[thread overview]
Message-ID: <E1RSjTJ-0007yO-Hd@fencepost.gnu.org> (raw)
In-Reply-To: <6D358885-0CD8-4FC0-9598-549AB7E3D4AC@gmail.com> (message from David Reitter on Mon, 21 Nov 2011 18:50:07 -0500)

> From: David Reitter <david.reitter@gmail.com>
> Date: Mon, 21 Nov 2011 18:50:07 -0500
> Cc: Eli Zaretskii <eliz@gnu.org>,
>  emacs-devel@gnu.org
> 
> On Nov 20, 2011, at 7:19 PM, YAMAMOTO Mitsuharu wrote:
> >  Then, the second line is displayed as `b' instead of `2'.
> 
> Because nobody else replied:  I reproduced it easily and was shocked too see this.

I have no reasons to believe the test case will not reproduce the
problem.  The reply that is expected from me is whether the proposed
patch is correct or not, and for that it is not enough to confirm that
the test case is reproducible.  I don't have the answer yet.

FWIW, this code was not touched since Emacs 21.1 was released.  So
evidently the effects of this issue are quite subtle in practice.

> Please do fix this bug before the next release.

You can help if you explain in more detail what is wrong with the
current code.  That is what I need to assess the correctness of the
patch.  I hoped that Yamamoto-san would do that.  Failing that, I'll
need to figure it out myself, which will take more time, and thus will
have to wait a couple of days until I have that time to sit down and
look into this.  I do intend to do that, make no mistake.

> I have, lately, seen an occasional and elusive problem with the display of header lines (or the lack thereof) and am now wondering if this could be due to the same issue.  I'll wait and see.

It would be better to describe the problem you are seeing, if not a
recipe to reproduce it.

Alternatively, you could apply the patch and see if those problems
disappear.



  reply	other threads:[~2011-11-22  6:04 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-11-20  7:13 Truncating scroll runs that copy to where we copied to YAMAMOTO Mitsuharu
2011-11-20 18:23 ` Eli Zaretskii
2011-11-21  0:19   ` YAMAMOTO Mitsuharu
2011-11-21 23:50     ` David Reitter
2011-11-22  6:04       ` Eli Zaretskii [this message]
2011-11-22  6:22         ` YAMAMOTO Mitsuharu
2011-11-22  8:25           ` Eli Zaretskii
2011-11-22  8:47             ` YAMAMOTO Mitsuharu
2011-11-22  7:26         ` YAMAMOTO Mitsuharu
2011-11-22  8:52           ` Eli Zaretskii
2011-11-22  9:09             ` YAMAMOTO Mitsuharu
2011-11-22  9:54               ` Eli Zaretskii
2011-11-23  0:41                 ` YAMAMOTO Mitsuharu
2011-11-26 12:44                   ` Eli Zaretskii
2011-11-28  1:10                     ` YAMAMOTO Mitsuharu
2011-11-22  0:33 ` YAMAMOTO Mitsuharu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=E1RSjTJ-0007yO-Hd@fencepost.gnu.org \
    --to=eliz@gnu.org \
    --cc=david.reitter@gmail.com \
    --cc=emacs-devel@gnu.org \
    --cc=mituharu@math.s.chiba-u.ac.jp \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).