From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!.POSTED!not-for-mail From: Noam Postavsky Newsgroups: gmane.emacs.bugs Subject: bug#28997: 27.0.50; Error in async shell-command process filter Date: Tue, 31 Oct 2017 06:48:52 -0400 Message-ID: <87wp3ba8yz.fsf@users.sourceforge.net> References: <8737672pna.fsf@gmail.com> NNTP-Posting-Host: blaine.gmane.org Mime-Version: 1.0 Content-Type: text/plain X-Trace: blaine.gmane.org 1509447494 22241 195.159.176.226 (31 Oct 2017 10:58:14 GMT) X-Complaints-To: usenet@blaine.gmane.org NNTP-Posting-Date: Tue, 31 Oct 2017 10:58:14 +0000 (UTC) User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/26.0.90 (gnu/linux) Cc: 28997@debbugs.gnu.org To: "Basil L. Contovounesios" Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Tue Oct 31 11:58:09 2017 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 1e9UFC-0004yT-DM for geb-bug-gnu-emacs@m.gmane.org; Tue, 31 Oct 2017 11:58:06 +0100 Original-Received: from localhost ([::1]:44734 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1e9UFH-00063z-TD for geb-bug-gnu-emacs@m.gmane.org; Tue, 31 Oct 2017 06:58:11 -0400 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:58790) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1e9U7T-0007AO-MW for bug-gnu-emacs@gnu.org; Tue, 31 Oct 2017 06:50:08 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1e9U7Q-0006r0-Jl for bug-gnu-emacs@gnu.org; Tue, 31 Oct 2017 06:50:07 -0400 Original-Received: from debbugs.gnu.org ([208.118.235.43]:34584) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1e9U7Q-0006qi-Fo for bug-gnu-emacs@gnu.org; Tue, 31 Oct 2017 06:50:04 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1e9U7O-0007L5-5E for bug-gnu-emacs@gnu.org; Tue, 31 Oct 2017 06:50:04 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Noam Postavsky Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Tue, 31 Oct 2017 10:50:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 28997 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: patch Original-Received: via spool by 28997-submit@debbugs.gnu.org id=B28997.150944694428144 (code B ref 28997); Tue, 31 Oct 2017 10:50:02 +0000 Original-Received: (at 28997) by debbugs.gnu.org; 31 Oct 2017 10:49:04 +0000 Original-Received: from localhost ([127.0.0.1]:43264 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1e9U6S-0007Jr-8E for submit@debbugs.gnu.org; Tue, 31 Oct 2017 06:49:04 -0400 Original-Received: from mail-io0-f182.google.com ([209.85.223.182]:52661) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1e9U6O-0007JM-2v for 28997@debbugs.gnu.org; Tue, 31 Oct 2017 06:49:00 -0400 Original-Received: by mail-io0-f182.google.com with SMTP id f20so33998061ioj.9 for <28997@debbugs.gnu.org>; Tue, 31 Oct 2017 03:48:59 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=sender:from:to:cc:subject:references:date:in-reply-to:message-id :user-agent:mime-version; bh=v8qDiY7RK+m6hh8KLMz800KTuORMrFI5WGg2SOSxqHA=; b=DDXIeYBFlY8EHKQSCOG3otSAXDwU123Cs63kP4zRAkCn+hL1NbHHEO1lDER79mgsD8 K7NNVw+JupSjdVwkEJi4ykT0VIjljprFpg+CJXHzlOCf+DVfp3X87fosmn9CeVcbh9eB OK2oL2txYWbEb51EZ1s62T1W59lvLMY/n58+VAX2gifokCZZQsi3WUXTJi5gCZVpXyK7 mTUeJVOXiidyLxyP3ofbCDUafzwXOfOFHacz8LgetH5Nk6r72RZx1bosMS34qhV/T5Uv jBKY3eP8Z0yqxTUigK8Ulw4Eom1xfGeaG9dTsPXKC99c+Zq9MV82uPPVDJ2AhAHe1NoJ AFrw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:sender:from:to:cc:subject:references:date :in-reply-to:message-id:user-agent:mime-version; bh=v8qDiY7RK+m6hh8KLMz800KTuORMrFI5WGg2SOSxqHA=; b=KLdeEQNVqLm+cbKgRw7LaAc/QrIgbR5YmgT12uSeH2GDLxhwonBGagAN85JZmoUW4p UccasqCCp4vtfhqy11071G2A+cm7mkObeNYeBPUEj9F77Mu+eEfvauTOioeCsalu73lI wHMJT3qeynzfB62E0f0ub5qofZSftVQBUHRe320P+bTX14DkJs4HRXxtS+bEjZ7eE2mb HRl+B3Hsi4bKVgItESofVV6rcOQsDHG0iQbWnl6fi2hLTEGtBFxa+EAnZVX8VjtG6DSq 164Wzp/Lz35JwklrvqBnXFpRv38va0HkYuwINUiYM9uD/zGz+k9AqD38SjK86XTu7pYB TDQw== X-Gm-Message-State: AMCzsaVm5CMRYqDwyxgBBVX5OGMSZAN9lW3oNJ1wIBCJTECukUvF7a10 a2K2xh3+fj0o64rWTQwYdOP2kg== X-Google-Smtp-Source: ABhQp+Sev2mWmVKoVsmcmVNNs/P9b5iHwwxf8ZNJiet2p03oxi21/wwwG9zsytHyQrWaZ8JJspCnWA== X-Received: by 10.36.67.141 with SMTP id s135mr2514255itb.142.1509446934208; Tue, 31 Oct 2017 03:48:54 -0700 (PDT) Original-Received: from zebian ([45.2.119.34]) by smtp.googlemail.com with ESMTPSA id q94sm541381ioi.18.2017.10.31.03.48.52 (version=TLS1_2 cipher=ECDHE-RSA-CHACHA20-POLY1305 bits=256/256); Tue, 31 Oct 2017 03:48:53 -0700 (PDT) In-Reply-To: <8737672pna.fsf@gmail.com> (Basil L. Contovounesios's message of "Wed, 25 Oct 2017 16:50:01 +0100") 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:139252 Archived-At: "Basil L. Contovounesios" writes: > The first patch replaces the quoted lambda process filter > with a closure over the name of output-buffer. By keeping > track of this name, a lot of the surrounding > buffer-name-determining code can be simplified via DRY. This looks good to me. > The second patch is purely aesthetic and simplifies the cond > that is touched by my first patch into an equivalent pcase. I'm not sure about this one. There was some contention in the past over use of pcase, with some suggestions even to make an opposite transformation, e.g., [1]. [1]: https://lists.gnu.org/archive/html/emacs-devel/2016-01/msg00055.html