From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Lennart Borgman Newsgroups: gmane.emacs.devel Subject: Re: The unwarranted scrolling assumption Date: Sat, 19 Jun 2010 22:38:24 +0200 Message-ID: References: <83iq5hiiin.fsf@gnu.org> <83fx0lihov.fsf@gnu.org> <838w6cixma.fsf@gnu.org> <83y6ech1oo.fsf@gnu.org> <83ocf8gx7e.fsf@gnu.org> <83fx0jgxk7.fsf@gnu.org> <83eig3gspa.fsf@gnu.org> <83bpb7gp2g.fsf@gnu.org> <837hlvglvh.fsf@gnu.org> <834ogzglaj.fsf@gnu.org> <8339wjgg8w.fsf@gnu.org> <83y6eagb7g.fsf@gnu.org> <4C1D24E3.5080402@harpegolden.net> NNTP-Posting-Host: lo.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 X-Trace: dough.gmane.org 1276979943 11662 80.91.229.12 (19 Jun 2010 20:39:03 GMT) X-Complaints-To: usenet@dough.gmane.org NNTP-Posting-Date: Sat, 19 Jun 2010 20:39:03 +0000 (UTC) Cc: emacs-devel@gnu.org, David De La Harpe Golden To: Juanma Barranquero Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Sat Jun 19 22:39:02 2010 connect(): No such file or directory Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([199.232.76.165]) by lo.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1OQ4od-0007pW-TV for ged-emacs-devel@m.gmane.org; Sat, 19 Jun 2010 22:39:00 +0200 Original-Received: from localhost ([127.0.0.1]:46582 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1OQ4od-0000jX-6C for ged-emacs-devel@m.gmane.org; Sat, 19 Jun 2010 16:38:59 -0400 Original-Received: from [140.186.70.92] (port=36147 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1OQ4oU-0000ik-Cb for emacs-devel@gnu.org; Sat, 19 Jun 2010 16:38:51 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1OQ4oP-0000ui-6t for emacs-devel@gnu.org; Sat, 19 Jun 2010 16:38:50 -0400 Original-Received: from mail-gy0-f169.google.com ([209.85.160.169]:44941) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1OQ4oP-0000ub-1f for emacs-devel@gnu.org; Sat, 19 Jun 2010 16:38:45 -0400 Original-Received: by gyg4 with SMTP id 4so2055570gyg.0 for ; Sat, 19 Jun 2010 13:38:44 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:mime-version:received:in-reply-to :references:from:date:message-id:subject:to:cc:content-type; bh=r4qc4SZxUf7Jv/KM8C0j/oUNklBvINtMX6gMX1h7wM4=; b=pkkLcf3pyE5SHwYbUOwLvda61QK3QdVGFfgg3P2N1gxezQo9YJzDSTlT25TH5yHOb5 syRGUxg3pzi7eAuceU93YTAYGt6d67joT2UyuOaGpCWuZrk3jLMJKnd0H96zH7s3kZ7f Ht47mpSl+GGasklsscSKWmd+JvJjlPN2+jXX4= DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc:content-type; b=KQin8+T9AqdjhDSXfHfvwQc9hOd4EAknHoyvlkr7bEex7B3+V/H1wtyCd6z5FQ6EHy TvUiFheJfswnoVSB3vn4Tbo1CmzNZoiKR9ovD4voVeNHeMHtdaIkpxlPdpQtDvASgX8z 6NoHWqeHga/cHzOcsOLz4sKeyAAJBMhikryBI= Original-Received: by 10.101.151.26 with SMTP id d26mr2273829ano.190.1276979924145; Sat, 19 Jun 2010 13:38:44 -0700 (PDT) Original-Received: by 10.100.154.15 with HTTP; Sat, 19 Jun 2010 13:38:24 -0700 (PDT) In-Reply-To: X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.6 (newer, 2) X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.5 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:126251 Archived-At: On Sat, Jun 19, 2010 at 10:30 PM, Juanma Barranquero wrote: > On Sat, Jun 19, 2010 at 22:26, Lennart Borgman > wrote: > >> This is expected. The original problem is still only addressed in my patch. >> >> And that is not applied. > > Then, help Eli understand why your patch is the right fix. I have done the best I can. I do not know what to do. The basic assumption that clip_changed can be set in narrow_to_region etc and that redisplay later can decide when it can be reset is false. Redisplay does not have the necessary information since it does not know exactly why clip_changed was set to 1. I have said that there is no reason to let narrow_to_region set clip_changed (even if it really seems intuitively right to do so). It is much easier to let redisplay do that instead in reconsider_clip_changes because then all needed information for the specific window is available. Is this clear?