From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!.POSTED!not-for-mail From: Stefan Monnier Newsgroups: gmane.emacs.devel Subject: Re: wait_reading_process_ouput hangs in certain cases (w/ patches) Date: Tue, 07 Nov 2017 12:23:18 -0500 Message-ID: References: <83lgjz8eiy.fsf@gnu.org> <831slp98ut.fsf@gnu.org> <83tvyj62qg.fsf@gnu.org> <83r2tetf90.fsf@gnu.org> <5150d198-8dd3-9cf4-5914-b7e945294452@binary-island.eu> <83tvy7s6wi.fsf@gnu.org> NNTP-Posting-Host: blaine.gmane.org Mime-Version: 1.0 Content-Type: text/plain X-Trace: blaine.gmane.org 1510075903 27884 195.159.176.226 (7 Nov 2017 17:31:43 GMT) X-Complaints-To: usenet@blaine.gmane.org NNTP-Posting-Date: Tue, 7 Nov 2017 17:31:43 +0000 (UTC) User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/27.0.50 (gnu/linux) To: emacs-devel@gnu.org Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Tue Nov 07 18:31:38 2017 Return-path: Envelope-to: ged-emacs-devel@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 1eC7io-0006xG-Od for ged-emacs-devel@m.gmane.org; Tue, 07 Nov 2017 18:31:34 +0100 Original-Received: from localhost ([::1]:54731 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1eC7iw-0007hd-3n for ged-emacs-devel@m.gmane.org; Tue, 07 Nov 2017 12:31:42 -0500 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:54158) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1eC7ck-0002Id-50 for emacs-devel@gnu.org; Tue, 07 Nov 2017 12:25:24 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1eC7cg-0005TU-Uo for emacs-devel@gnu.org; Tue, 07 Nov 2017 12:25:18 -0500 Original-Received: from [195.159.176.226] (port=39377 helo=blaine.gmane.org) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1eC7cg-0005T4-Ja for emacs-devel@gnu.org; Tue, 07 Nov 2017 12:25:14 -0500 Original-Received: from list by blaine.gmane.org with local (Exim 4.84_2) (envelope-from ) id 1eC7cV-0008Ui-Ek for emacs-devel@gnu.org; Tue, 07 Nov 2017 18:25:03 +0100 X-Injected-Via-Gmane: http://gmane.org/ Original-Lines: 17 Original-X-Complaints-To: usenet@blaine.gmane.org Cancel-Lock: sha1:SApIlS4K4a6WjNWq9ZT0IyMSxvE= X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] [fuzzy] X-Received-From: 195.159.176.226 X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.21 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:219969 Archived-At: > /* Byte-count for process output read from `infd'. */ > unsigned long infd_num_bytes_read; BTW, we could count the number of (non-empty) "chunks" rather than the number of bytes. > { > got_some_output = 1; > break; > } Please try to use `true' and `false' for boolean values (there's still a lot of code in src/*.c which uses 0 and 1, admittedly, but this should slowly disappear over time). Stefan