From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!.POSTED!not-for-mail From: Reuben Thomas Newsgroups: gmane.emacs.bugs Subject: bug#18133: Suppressing asynchronous command output Date: Tue, 27 Dec 2016 01:09:23 +0000 Message-ID: References: <83zijp180n.fsf@gnu.org> <83eg100vy5.fsf@gnu.org> <585C132B.1030709@gmx.at> <585C347D.9050309@gmx.at> <585D740B.40303@gmx.at> <585D8120.1090300@gmx.at> <87fula1dbk.fsf@mail.linkov.net> NNTP-Posting-Host: blaine.gmane.org Mime-Version: 1.0 Content-Type: multipart/mixed; boundary=94eb2c085152bda40c05449980fa X-Trace: blaine.gmane.org 1482801020 3197 195.159.176.226 (27 Dec 2016 01:10:20 GMT) X-Complaints-To: usenet@blaine.gmane.org NNTP-Posting-Date: Tue, 27 Dec 2016 01:10:20 +0000 (UTC) Cc: 18133@debbugs.gnu.org To: Juri Linkov Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Tue Dec 27 02:10:14 2016 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by blaine.gmane.org with esmtp (Exim 4.84_2) (envelope-from ) id 1cLgHN-0007zO-Bp for geb-bug-gnu-emacs@m.gmane.org; Tue, 27 Dec 2016 02:10:13 +0100 Original-Received: from localhost ([::1]:52285 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1cLgHO-0002Mz-TR for geb-bug-gnu-emacs@m.gmane.org; Mon, 26 Dec 2016 20:10:14 -0500 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:45883) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1cLgHF-0002LV-JP for bug-gnu-emacs@gnu.org; Mon, 26 Dec 2016 20:10:06 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1cLgHC-0007KR-Am for bug-gnu-emacs@gnu.org; Mon, 26 Dec 2016 20:10:05 -0500 Original-Received: from debbugs.gnu.org ([208.118.235.43]:40773) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1cLgHC-0007KG-7S for bug-gnu-emacs@gnu.org; Mon, 26 Dec 2016 20:10:02 -0500 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1cLgHB-0005Sn-SQ for bug-gnu-emacs@gnu.org; Mon, 26 Dec 2016 20:10:01 -0500 X-Loop: help-debbugs@gnu.org Resent-From: Reuben Thomas Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Tue, 27 Dec 2016 01:10:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 18133 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: Original-Received: via spool by 18133-submit@debbugs.gnu.org id=B18133.148280097220956 (code B ref 18133); Tue, 27 Dec 2016 01:10:01 +0000 Original-Received: (at 18133) by debbugs.gnu.org; 27 Dec 2016 01:09:32 +0000 Original-Received: from localhost ([127.0.0.1]:56172 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1cLgGh-0005Rw-NA for submit@debbugs.gnu.org; Mon, 26 Dec 2016 20:09:32 -0500 Original-Received: from mail-qk0-f171.google.com ([209.85.220.171]:34441) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1cLgGf-0005Ri-QR for 18133@debbugs.gnu.org; Mon, 26 Dec 2016 20:09:30 -0500 Original-Received: by mail-qk0-f171.google.com with SMTP id h201so68634230qke.1 for <18133@debbugs.gnu.org>; Mon, 26 Dec 2016 17:09:29 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=sc3d.org; s=google; h=mime-version:in-reply-to:references:from:date:message-id:subject:to :cc; bh=Ldwvvp30zyoAJ/GLy2pFnyRA/IV17blGPeUfGM2x8MQ=; b=JV7gW0bcQz/VTg+euQaYuNCsf+52uapd4hqetxyPcuN5L4rpprQNWEZO1WlUypuq9F p0OCfR9rwMfb4rCxe/nQ4FzmxsUtesbjwCGBb/Hr+E1/aspfkyDmIAa8rDKdirhF7N1k tKb83I1FtGSXlAIA0iEFp0Vunv8yW/SEtTqZ4= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:in-reply-to:references:from:date :message-id:subject:to:cc; bh=Ldwvvp30zyoAJ/GLy2pFnyRA/IV17blGPeUfGM2x8MQ=; b=gEFVhW6+xdDyzVFzwVpQNlLXYlUy2Lklhiyr8VwuPBPX24ZqU3Kdo67QKfHuUSAJr+ ZQqW+cEcGk2OIJJXJygNROe3Yq8arWzP5aR7Kac9LlYs9e4YkQ1tRfAdzXbRa2nEy+U+ Ik2tDvZ/AV8ouyZtQDMLtNIvR2s382uoPYEFLuzxfEHcQTkzkwEYuH13tpFFNkus/DWU 8hFEhBLJN9DqkcFv80AwOmYVrMQbaSefNzW5POm7QYBRDtBQ+ePLketug1BHloSUE6YL LUyk4Y56c4623iJqzvab4xGYkHdiAVwnQ2BQMYG5GXnET6CuNdDu5ROcosI6vPXUD2dz mXag== X-Gm-Message-State: AIkVDXKPXnpmw5Y4vsnhusoHeTbiyrdjmNaiHxSFf2fe6MF+Lpn8PIHIqjD8e+n8laVvuOPna6A7HkjBrX6szkmM X-Received: by 10.55.148.199 with SMTP id w190mr14421532qkd.9.1482800964130; Mon, 26 Dec 2016 17:09:24 -0800 (PST) Original-Received: by 10.140.88.51 with HTTP; Mon, 26 Dec 2016 17:09:23 -0800 (PST) In-Reply-To: <87fula1dbk.fsf@mail.linkov.net> X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 208.118.235.43 X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Original-Sender: "bug-gnu-emacs" Xref: news.gmane.org gmane.emacs.bugs:127474 Archived-At: --94eb2c085152bda40c05449980fa Content-Type: multipart/alternative; boundary=94eb2c085152bda40805449980f8 --94eb2c085152bda40805449980f8 Content-Type: text/plain; charset=UTF-8 On 26 December 2016 at 23:27, Juri Linkov wrote: [message cut] Thanks, Juri, for helping again with this bug. I attach an updated version of my patch, which uses an :options setting for display-buffer-alist so that the current behaviour remains the default, and runs a preoutput-filter-function which calls display-buffer only when output is added to an empty buffer. I believe this addresses the performance and behaviour issues, without the need for another user option. It is also rather shorter than the previous patch. -- http://rrt.sc3d.org --94eb2c085152bda40805449980f8 Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: quoted-printable
On = 26 December 2016 at 23:27, Juri Linkov <juri@linkov.net> wrote= :

[message cut]
=

Thanks, Juri, for helping agai= n with this bug.

I attach= an updated version of my patch, which uses an :options setting for display= -buffer-alist so that the current behaviour remains the default, and runs a= preoutput-filter-function which calls display-buffer only when output is a= dded to an empty buffer.

I believe this addresses the performance and behaviour issues, without = the need for another user option.

It is also rather shorter than the previous patch.

--
http://rrt.sc3d.org
--94eb2c085152bda40805449980f8-- --94eb2c085152bda40c05449980fa Content-Type: text/x-patch; charset=US-ASCII; name="0001-Delay-showing-Async-Shell-Command-buffer-until-outpu.patch" Content-Disposition: attachment; filename="0001-Delay-showing-Async-Shell-Command-buffer-until-outpu.patch" Content-Transfer-Encoding: base64 X-Attachment-Id: f_ix6te0wn0 RnJvbSA5MzVkNTBkOTNkMzdkMmQ0NjQ5NjczYTVkNTJlYzRlZmU0YWJjZTdiIE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBSZXViZW4gVGhvbWFzIDxycnRAc2MzZC5vcmc+CkRhdGU6IE1v biwgMTkgRGVjIDIwMTYgMTU6Mzg6MzYgKzAwMDAKU3ViamVjdDogW1BBVENIXSBEZWxheSBzaG93 aW5nIEFzeW5jIFNoZWxsIENvbW1hbmQgYnVmZmVyIHVudGlsIG91dHB1dAogKEJ1ZyMxODEzMykK CiogbGlzcC9jb21pbnQuZWwgKGNvbWludC1pbnB1dC1maWx0ZXIpOiBBZGQgcHJvY2VzcyBhcmd1 bWVudC4KKGNvbWludC1vdXRwdXQtZmlsdGVyLWZ1bmN0aW9ucyk6IEFkZCBgY29taW50LW1ha2Ut YnVmZmVyLXZpc2libGUnIHRvCnRoZSBsaXN0LCBkb2N1bWVudCB0aGUgc2Vjb25kIGFyZ3VtZW50 ICh0aGUgcHJvY2VzcykuCihjb21pbnQtc2VuZC1pbnB1dCk6IENhbGwgaW5wdXQgYW5kIG91dHB1 dCBmaWx0ZXIgZnVuY3Rpb25zIHdpdGgKcHJvY2VzcyBhcmd1bWVudC4KKGNvbWludC1vdXRwdXQt ZmlsdGVyKTogQ2FsbCBvdXRwdXQgZmlsdGVyIGZ1bmN0aW9ucyB3aXRoIHByb2Nlc3MKYXJndW1l bnQuCihjb21pbnQtbWFrZS1idWZmZXItdmlzaWJsZSk6IE5ldyBob29rIGZ1bmN0aW9uLCB0byBt YWtlIEFzeW5jIFNoZWxsCkNvbW1hbmQgYnVmZmVyIHZpc2libGUuCihjb21pbnQtcG9zdG91dHB1 dC1zY3JvbGwtdG8tYm90dG9tKToKKGNvbWludC13YXRjaC1mb3ItcGFzc3dvcmQtcHJvbXB0KTog QWRkIGR1bW15IHByb2Nlc3MgYXJndW1lbnQuCiogbGlzcC93aW5kb3cuZWwgKGRpc3BsYXktYnVm ZmVyLWFsaXN0KTogQWRkIGEgZGVmYXVsdCBlbnRyeSB0byBoaWRlCkFzeW5jIFNoZWxsIENvbW1h bmQgb3V0cHV0IHdpbmRvdy4KKiBsaXNwL2Fuc2ktY29sb3IuZWwgKGFuc2ktY29sb3ItcHJvY2Vz cy1vdXRwdXQpOiBBZGQgYSBkdW1teSBwcm9jZXNzCmFyZ3VtZW50LgotLS0KIGxpc3AvY29taW50 LmVsIHwgMjIgKysrKysrKysrKysrKysrKy0tLS0tLQogbGlzcC93aW5kb3cuZWwgfCAgNSArKysr LQogMiBmaWxlcyBjaGFuZ2VkLCAyMCBpbnNlcnRpb25zKCspLCA3IGRlbGV0aW9ucygtKQoKZGlm ZiAtLWdpdCBhL2xpc3AvY29taW50LmVsIGIvbGlzcC9jb21pbnQuZWwKaW5kZXggYjljNjViMC4u NmZhMmZjOSAxMDA2NDQKLS0tIGEvbGlzcC9jb21pbnQuZWwKKysrIGIvbGlzcC9jb21pbnQuZWwK QEAgLTE5MTMsMTIgKzE5MTMsMTIgQEAgY29taW50LXNlbmQtaW5wdXQKICAgICAgICAgOzsgYnV0 IHRoYXQgc2Nyb2xsZWQgdGhlIGJ1ZmZlciBpbiB1bmRlc2lyYWJsZSB3YXlzLgogICAgICAgICAo cnVuLWhvb2std2l0aC1hcmdzICdjb21pbnQtb3V0cHV0LWZpbHRlci1mdW5jdGlvbnMgIiIpKSkp KQogCi0oZGVmdmFyIGNvbWludC1wcmVvdXRwdXQtZmlsdGVyLWZ1bmN0aW9ucyBuaWwKKyhkZWZ2 YXIgY29taW50LXByZW91dHB1dC1maWx0ZXItZnVuY3Rpb25zICcoY29taW50LW1ha2UtbmV3bHkt d3JpdHRlbi1idWZmZXItdmlzaWJsZSkKICAgIkxpc3Qgb2YgZnVuY3Rpb25zIHRvIGNhbGwgYmVm b3JlIGluc2VydGluZyBDb21pbnQgb3V0cHV0IGludG8gdGhlIGJ1ZmZlci4KLUVhY2ggZnVuY3Rp b24gZ2V0cyBvbmUgYXJndW1lbnQsIGEgc3RyaW5nIGNvbnRhaW5pbmcgdGhlIHRleHQgcmVjZWl2 ZWQKLWZyb20gdGhlIHN1YnByb2Nlc3MuICBJdCBzaG91bGQgcmV0dXJuIHRoZSBzdHJpbmcgdG8g aW5zZXJ0LCBwZXJoYXBzCi10aGUgc2FtZSBzdHJpbmcgdGhhdCB3YXMgcmVjZWl2ZWQsIG9yIHBl cmhhcHMgYSBtb2RpZmllZCBvciB0cmFuc2Zvcm1lZAotc3RyaW5nLgorRWFjaCBmdW5jdGlvbiBn ZXRzIHR3byBhcmd1bWVudCwgYSBzdHJpbmcgY29udGFpbmluZyB0aGUgdGV4dCByZWNlaXZlZAor ZnJvbSB0aGUgc3VicHJvY2VzcywgYW5kIHRoZSBwcm9jZXNzLiAgSXQgc2hvdWxkIHJldHVybiB0 aGUgc3RyaW5nIHRvCitpbnNlcnQsIHBlcmhhcHMgdGhlIHNhbWUgc3RyaW5nIHRoYXQgd2FzIHJl Y2VpdmVkLCBvciBwZXJoYXBzIGEgbW9kaWZpZWQgb3IKK3RyYW5zZm9ybWVkIHN0cmluZy4KIAog VGhlIGZ1bmN0aW9ucyBvbiB0aGUgbGlzdCBhcmUgY2FsbGVkIHNlcXVlbnRpYWxseSwgYW5kIGVh Y2ggb25lIGlzCiBnaXZlbiB0aGUgc3RyaW5nIHJldHVybmVkIGJ5IHRoZSBwcmV2aW91cyBvbmUu ICBUaGUgc3RyaW5nIHJldHVybmVkIGJ5CkBAIC0yMDIzLDcgKzIwMjMsNyBAQCBjb21pbnQtb3V0 cHV0LWZpbHRlcgogCQkobGV0ICgoZnVuY3Rpb25zCiAgICAgICAgICAgICAgICAgICAgICAgIChk ZWZhdWx0LXZhbHVlICdjb21pbnQtcHJlb3V0cHV0LWZpbHRlci1mdW5jdGlvbnMpKSkKIAkJICAo d2hpbGUgKGFuZCBmdW5jdGlvbnMgc3RyaW5nKQotCQkgICAgKHNldHEgc3RyaW5nIChmdW5jYWxs IChjYXIgZnVuY3Rpb25zKSBzdHJpbmcpKQorCQkgICAgKHNldHEgc3RyaW5nIChmdW5jYWxsIChj YXIgZnVuY3Rpb25zKSBzdHJpbmcgcHJvY2VzcykpCiAJCSAgICAoc2V0cSBmdW5jdGlvbnMgKGNk ciBmdW5jdGlvbnMpKSkpCiAJICAgICAgKHNldHEgc3RyaW5nIChmdW5jYWxsIChjYXIgZnVuY3Rp b25zKSBzdHJpbmcpKSkKIAkgICAgKHNldHEgZnVuY3Rpb25zIChjZHIgZnVuY3Rpb25zKSkpKQpA QCAtMjExMSw2ICsyMTExLDE2IEBAIGNvbWludC1vdXRwdXQtZmlsdGVyCiAJICAgICAgKGFkZC10 ZXh0LXByb3BlcnRpZXMgcHJvbXB0LXN0YXJ0IChwb2ludCkgJyhyZWFyLW5vbnN0aWNreSB0KSkp CiAJICAgIChnb3RvLWNoYXIgc2F2ZWQtcG9pbnQpKSkpKSkpCiAKKyhkZWZ1biBjb21pbnQtbWFr ZS1uZXdseS13cml0dGVuLWJ1ZmZlci12aXNpYmxlIChzdHJpbmcgcHJvY2VzcykKKyAgIk1ha2Ug dGhlIG91dHB1dCBidWZmZXIgdmlzaWJsZSB3aGVuIG91dHB1dCBpcyBhZGRlZCB0byBhbiBlbXB0 eSBidWZmZXIuCitVc2VmdWwgaW4gYGNvbWludC1wcmVvdXRwdXQtZmlsdGVyLWZ1bmN0aW9ucycu IgorICAobGV0ICgoYnVmZmVyIChwcm9jZXNzLWJ1ZmZlciBwcm9jZXNzKSkpCisgICAgKHdoZW4g KGFuZCAoPSAwIChidWZmZXItc2l6ZSBidWZmZXIpKQorICAgICAgICAgICAgICAgKHN0cmluZy1t YXRjaC1wICJcXCpBc3luYyBTaGVsbCBDb21tYW5kXFwqIgorICAgICAgICAgICAgICAgICAgICAg ICAgICAgICAgIChidWZmZXItbmFtZSBidWZmZXIpKSkKKyAgICAgIChkaXNwbGF5LWJ1ZmZlciAo cHJvY2Vzcy1idWZmZXIgcHJvY2VzcykpKSkKKyAgc3RyaW5nKQorCiAoZGVmdW4gY29taW50LXBy ZWlucHV0LXNjcm9sbC10by1ib3R0b20gKCkKICAgIkdvIHRvIHRoZSBlbmQgb2YgYnVmZmVyIGlu IGFsbCB3aW5kb3dzIHNob3dpbmcgaXQuCiBNb3ZlbWVudCBvY2N1cnMgaWYgcG9pbnQgaW4gdGhl IHNlbGVjdGVkIHdpbmRvdyBpcyBub3QgYWZ0ZXIgdGhlIHByb2Nlc3MgbWFyaywKZGlmZiAtLWdp dCBhL2xpc3Avd2luZG93LmVsIGIvbGlzcC93aW5kb3cuZWwKaW5kZXggZmRiNjdlZC4uYTQ5ZjRh MyAxMDA2NDQKLS0tIGEvbGlzcC93aW5kb3cuZWwKKysrIGIvbGlzcC93aW5kb3cuZWwKQEAgLTY3 OTcsNyArNjc5NywxMCBAQCBkaXNwbGF5LWJ1ZmZlci1hbGlzdAogCQkoY2hvaWNlIDp0YWcgIkNv bmRpdGlvbiIKIAkJCXJlZ2V4cAogCQkJKGZ1bmN0aW9uIDp0YWcgIk1hdGNoZXIgZnVuY3Rpb24i KSkKLQkJOnZhbHVlLXR5cGUgLGRpc3BsYXktYnVmZmVyLS1hY3Rpb24tY3VzdG9tLXR5cGUpCisJ CTp2YWx1ZS10eXBlICxkaXNwbGF5LWJ1ZmZlci0tYWN0aW9uLWN1c3RvbS10eXBlCisJCTpvcHRp b25zICgoIlxcKkFzeW5jIFNoZWxsIENvbW1hbmRcXCoiCisgICAgICAgICAgICAgICAgICAgICAg ICAgICAoY29ucworICAgICAgICAgICAgICAgICAgICAgICAgICAgIChjb25zdCBkaXNwbGF5LWJ1 ZmZlci1uby13aW5kb3cpIChjb25zdCBuaWwpKSkpKQogICA6cmlza3kgdAogICA6dmVyc2lvbiAi MjQuMSIKICAgOmdyb3VwICd3aW5kb3dzKQotLSAKMi43LjQKCg== --94eb2c085152bda40c05449980fa--