From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!.POSTED.blaine.gmane.org!not-for-mail From: Juanma Barranquero Newsgroups: gmane.emacs.devel Subject: Re: warnings and window-point Date: Fri, 22 Nov 2019 09:43:43 +0100 Message-ID: References: <83lfs8wd2t.fsf@gnu.org> Mime-Version: 1.0 Content-Type: multipart/alternative; boundary="00000000000061ee700597eb6b5a" Injection-Info: blaine.gmane.org; posting-host="blaine.gmane.org:195.159.176.226"; logging-data="6545"; mail-complaints-to="usenet@blaine.gmane.org" Cc: Emacs developers To: Eli Zaretskii Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Fri Nov 22 09:46:41 2019 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([209.51.188.17]) by blaine.gmane.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.89) (envelope-from ) id 1iY4aO-0001Ue-Ig for ged-emacs-devel@m.gmane.org; Fri, 22 Nov 2019 09:46:40 +0100 Original-Received: from localhost ([::1]:48608 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iY4aN-0006Pg-EJ for ged-emacs-devel@m.gmane.org; Fri, 22 Nov 2019 03:46:39 -0500 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:47163) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1iY4Y9-0004Vl-Ha for emacs-devel@gnu.org; Fri, 22 Nov 2019 03:44:22 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1iY4Y8-0005Or-HC for emacs-devel@gnu.org; Fri, 22 Nov 2019 03:44:21 -0500 Original-Received: from mail-qk1-x735.google.com ([2607:f8b0:4864:20::735]:36536) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1iY4Y7-0005O6-5m; Fri, 22 Nov 2019 03:44:19 -0500 Original-Received: by mail-qk1-x735.google.com with SMTP id d13so5603000qko.3; Fri, 22 Nov 2019 00:44:19 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=fjcFlN7p1HgWzM2QiPXnJuWDYcPkBeStAV+RiAXV6lI=; b=Es1ZtlSGit59bKvxcNfkT2KAQFipSmVhTi5ukaeFl5A7u78sQMwBSNunEPBiXa2ksu 7xK0VY9mmt56zewXCgbbgJjzOnznSGwsvdxInNnJWKnHKorCqHusmi7WpjBCBJHKy9BG csKy0zUfl2Fq7BQoDd6nP/ugbkPade94+C0+x37TxMC7oF+9ba++E1w3zxtpu6QCUsqz AQuGaDnewqceW4vN1vZRugCEMWL5DNCZ1YOdBDyV4uSIN7MEpuPUUZpMjh/w5NvgYB/6 Jq269B4WuYG17tht2XeQ4dWsTMNCvgrbOu2lDAh8R/ttWB/3NyRLQfPuRqfvjjrFGEV8 OHQg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=fjcFlN7p1HgWzM2QiPXnJuWDYcPkBeStAV+RiAXV6lI=; b=HuNsMb2PCBoHZhn5dfhyOkuJnfvCm7cM/6FRKWXb3IMLqNoaNpIE8l9JinULpdVZIg jI2DyH4ZI5yJKL877RY6tHXbsnWHdURyJTIK7qt/EBhpX9l7ziuWzzyCCLE/oO7Oc/Tw EjaLhZcBkj85NJeBk4aaEPs6Dpi+j4i4o4b/+mNbEpk6gfHv3CMYwiYK74wsOk0deRlD mBB8RFI6Gi4MxhjurhyHxFNWhMzXx3C9iFvRytpd2Bc2QDDckoW3rBBhH//x5XmX0Od2 PiBvZIg3JmGXRhsAtGMGIn4mkA4gvG1dtvs/hLEYE7/5Fzw/NchNFPDA82qw/s9eCcAl DtuQ== X-Gm-Message-State: APjAAAWRv1keHKylkUK0PPNbc22ur3mNm3sTacVVDSXy5QE94EWrVXpQ 3Bue6rmS3jhqTnM8WSgg72uOIIJ+Ki/FO+UPZ/KMkQ== X-Google-Smtp-Source: APXvYqxgy6iz1DVZlbIqDL34MEcN7CHFl4Kp8ut8pVGqXmOLRmuCMa3iH8AvEb2YLw5Uv2bEE2SE9cmyFWS/kZoTnd8= X-Received: by 2002:a37:96c4:: with SMTP id y187mr6976963qkd.281.1574412258240; Fri, 22 Nov 2019 00:44:18 -0800 (PST) In-Reply-To: <83lfs8wd2t.fsf@gnu.org> X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 2607:f8b0:4864:20::735 X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Original-Sender: "Emacs-devel" Xref: news.gmane.org gmane.emacs.devel:242611 Archived-At: --00000000000061ee700597eb6b5a Content-Type: text/plain; charset="UTF-8" No, I don't propose to change it, at least not now. The problem I had that led me to study this I fixed locally with some advice. It's just curiosity, because I really don't understand the intent; particularly, the fact that the point is in the second warning is really weird. > > Perhaps the idea is to keep past warnings visible until the user scrolls the buffer themself Yeah, it's just that... As a justification isn't very good, because the buffer is autoscrolled if warning-series is set to non-nil. So it's like yeah, let's not move it, except in some cases we do. And warning-series is surely intended to be set by the application code generating the warnings, not the user, so when it does scroll, it doesn't do so under the user's direction or control. I honestly think that's really an oversight no one has either detected or, more likely, cared about until now because warnings.el doesn't get (I think) much use. So I'd like to try the patch above in the trunk after we branch for 27, just to see if someone complains. --00000000000061ee700597eb6b5a Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
No, I don't propose to change it, at least not now. Th= e problem I had that led me to study this I fixed locally with some advice.=

It's just curiosity, because I really don't understand the = intent; particularly, the fact that the point is in the second warning is r= eally weird.

> > Perhaps the idea is to keep past warnings vis= ible until the user scrolls the buffer themself

Yeah, it's just = that... As a justification isn't very good, because the buffer is autos= crolled if warning-series is set to non-nil. So it's like yeah, let'= ;s not move it, except in some cases we do. And warning-series is surely in= tended to be set by the application code generating the warnings, not the u= ser, so when it does scroll, it doesn't do so under the user's dire= ction or control.

I honestly think that's really an oversight no= one has either detected or, more likely, cared about until now because war= nings.el doesn't get (I think) much use.

So I'd like to try = the patch above in the trunk after we branch for 27, just to see if someone= complains.

--00000000000061ee700597eb6b5a--