From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 19243 invoked by alias); 24 Aug 2017 10:41:17 -0000 Mailing-List: contact zsh-workers-help@zsh.org; run by ezmlm Precedence: bulk X-No-Archive: yes List-Id: Zsh Workers List List-Post: List-Help: X-Seq: 41595 Received: (qmail 12771 invoked by uid 1010); 24 Aug 2017 10:41:17 -0000 X-Qmail-Scanner-Diagnostics: from mailout1.w1.samsung.com by f.primenet.com.au (envelope-from , uid 7791) with qmail-scanner-2.11 (clamdscan: 0.99.2/21882. spamassassin: 3.4.1. Clear:RC:0(210.118.77.11):SA:0(-6.9/5.0):. Processed in 2.448509 secs); 24 Aug 2017 10:41:17 -0000 X-Spam-Checker-Version: SpamAssassin 3.4.1 (2015-04-28) on f.primenet.com.au X-Spam-Level: X-Spam-Status: No, score=-6.9 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_HI, RCVD_IN_MSPIKE_H3,RCVD_IN_MSPIKE_WL,RP_MATCHES_RCVD,SPF_HELO_PASS,SPF_PASS autolearn=ham autolearn_force=no version=3.4.1 X-Envelope-From: p.stephenson@samsung.com X-Qmail-Scanner-Mime-Attachments: | X-Qmail-Scanner-Zip-Files: | X-AuditID: cbfec7f5-f79d06d0000031c7-a2-599eab7a7f86 Date: Thu, 24 Aug 2017 11:33:27 +0100 From: Peter Stephenson To: zsh-workers@zsh.org Subject: Re: Question about err_return Message-id: <20170824113327.2fab99b5@pwslap01u.europe.root.pri> In-reply-to: <20170823210951.6c430e82@ntlworld.com> Organization: Samsung Cambridge Solution Centre X-Mailer: Claws Mail 3.7.9 (GTK+ 2.22.0; i386-redhat-linux-gnu) MIME-version: 1.0 Content-type: text/plain; charset="US-ASCII" Content-transfer-encoding: 7bit X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFnrFIsWRmVeSWpSXmKPExsWy7djPc7pVq+dFGiw7KGJxsPkhkwOjx6qD H5gCGKO4bFJSczLLUov07RK4MjZ8XMhWsJWr4vLdL2wNjFM5uhg5OSQETCSOzvrHBmGLSVy4 tx7I5uIQEljKKLHy4xIWCKeXSWL/leOsMB1rj81ggkgsY5R492UVWLuQwDQmicVX6yHsM4wS VybxQhSdZZRomH4DrIhFQFXi3P5NYDabgKHE1E2zGUFsEQFxibNrz7OA2MICahJtG76ydzFy cPAK2Evs3JIKYnIKGEssvM4OUsEvoC9x9e8nJoh77CVmXjkDNoVXQFDix+R7YFOYBXQktm17 zA5hy0tsXvOWGeQcCYH/bBK7bt9gBZkpISArsekAM8QcF4lP3zqhfhSWeHV8CzuELSPR2XEQ alc/o8STbl+IOTMYJU6f2QENOmuJvtsXGSGW8UlM2jadGWI+r0RHmxBEiYfEiV1roModJU7/ mco8gVFxFpKzZyE5exaSsxcwMq9iFEktLc5NTy021StOzC0uzUvXS87P3cQITAGn/x3/uoNx 6TGrQ4wCHIxKPLwarfMihVgTy4orcw8xSnAwK4nwhlUAhXhTEiurUovy44tKc1KLDzFKc7Ao ifPaRrVFCgmkJ5akZqemFqQWwWSZODilGhjZnAvnZ3ww3vZL6cvimsPvVhz45mCRvOlx1a/u 5w1WXtP2uv7/UvXEVPlbVe/pe7/7laema1zW99qweXO52dZZE2NW5B7Q6oqpzqw68V/xtO/m fu67Qa8u2t27d8JgwZVt5eYORx3fmp1eK31jl/3OOVsY+fqzBbzmtdbcbdNK1FjzUHiLzetY JZbijERDLeai4kQADRXcQv0CAAA= X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFlrCIsWRmVeSWpSXmKPExsVy+t/xa7qVq+dFGhzZo29xsPkhkwOjx6qD H5gCGKPcbDJSE1NSixRS85LzUzLz0m2VQkPcdC2UFPISc1NtlSJ0fUOClBTKEnNKgTwjAzTg 4BzgHqykb5fglrHh40K2gq1cFZfvfmFrYJzK0cXIySEhYCKx9tgMJghbTOLCvfVsILaQwBJG icaPcV2MXED2DCaJ532trBDOOUaJrU//MEI4ZxklOr7PZQZpYRFQlTi3fxNYO5uAocTUTbMZ QWwRAXGJs2vPs4DYwgJqEm0bvrJ3MXJw8ArYS+zckgpicgoYSyy8zg4x8gejxP0LG8HK+QX0 Ja7+/QR1nb3EzCtnwEbyCghK/Jh8D6yGWUBLYvO2JlYIW15i85q3zBAfqEvcuLubfQKj8Cwk LbOQtMxC0rKAkXkVo0hqaXFuem6xoV5xYm5xaV66XnJ+7iZGYAxtO/Zz8w7GSxuDDzEKcDAq 8fBqtM6LFGJNLCuuzD3EKMHBrCTCG1YBFOJNSaysSi3Kjy8qzUktPsRoCgyXicxSosn5wPjO K4k3NDE0tzQ0MrawMDcyUhLnVb/cFCkkkJ5YkpqdmlqQWgTTx8TBKdXAaG3qIrgm4wdbWFBS nrCCc8Y+paK17w4fP7i7P3iZyUzlqNd6PYsaHnqdj3hVYXCrtljr7wrhb/wTPjpfLP+mflHr Xp/j3RV8XAe2+HgtWfWIwUn4T7za14bt72ean34jUXQ8xOG3y8zM3zuXcnPM8WjPnHjfSMJx wsF3QhKzf7xO/ffyo+PFOCWW4oxEQy3mouJEAGsZuye3AgAA X-MTR: 20000000000000000@CPGS X-CMS-MailID: 20170824103329eucas1p11fbcf976968589efb8b7d0dd37e226c6 X-Msg-Generator: CA X-Sender-IP: 182.198.249.179 X-Local-Sender: =?UTF-8?B?UGV0ZXIgU3RlcGhlbnNvbhtTQ1NDLURhdGEgUGxhbmUb?= =?UTF-8?B?7IK87ISx7KCE7J6QG1ByaW5jaXBhbCBFbmdpbmVlciwgU29mdHdhcmU=?= X-Global-Sender: =?UTF-8?B?UGV0ZXIgU3RlcGhlbnNvbhtTQ1NDLURhdGEgUGxhbmUbU2Ft?= =?UTF-8?B?c3VuZyBFbGVjdHJvbmljcxtQcmluY2lwYWwgRW5naW5lZXIsIFNvZnR3YXJl?= X-Sender-Code: =?UTF-8?B?QzEwG0VIURtDMTBDRDA1Q0QwNTAwNTg=?= CMS-TYPE: 201P X-CMS-RootMailID: 20170823201125epcas3p33003b4d4486735f75722e9f2e4e392d2 X-RootMTR: 20170823201125epcas3p33003b4d4486735f75722e9f2e4e392d2 References: <20170823175608.4e89cca4@ntlworld.com> <20170823210951.6c430e82@ntlworld.com> On Wed, 23 Aug 2017 21:09:51 +0100 Peter Stephenson wrote: > diff --git a/Src/init.c b/Src/init.c > index d8c26ac..9331b03 100644 > --- a/Src/init.c > +++ b/Src/init.c > @@ -1070,7 +1070,7 @@ setupvals(char *cmd, char *runscript, char *zsh_name) > sfcontext = SFC_NONE; > trap_return = 0; > trap_state = TRAP_STATE_INACTIVE; > - noerrexit = -1; > + noerrexit = NOERREXIT_SIGNAL; > nohistsave = 1; > dirstack = znewlinklist(); > bufstack = znewlinklist(); > @@ -1199,7 +1199,7 @@ init_signals(void) > void > run_init_scripts(void) > { > - noerrexit = -1; > + noerrexit = NOERREXIT_SIGNAL; > > if (EMULATION(EMULATE_KSH|EMULATE_SH)) { > if (islogin) I think the two assignments modified here need the standard flags as well since the intention is obviously to suppress error exit/return generally in startup files (not just modify the ^C behaviour, which is what the flag there at the moment does). Obviously err_return in functions called from initialisation files would get the new behaviour. That's probably sensible if your functions are designed to operate with the option. It's a little less obvious with anonymous functions, perhaps. Apart from that, I hope this isn't going to be problematic... pws