From: Peter Stephenson <pws@ibmth.df.unipi.it>
To: zsh-workers@sunsite.auc.dk (Zsh hackers list)
Subject: PATCH: pws-whatever: emulate -L sets localtraps
Date: Tue, 22 Jun 1999 17:47:02 +0200 [thread overview]
Message-ID: <9906221547.AA25847@ibmth.df.unipi.it> (raw)
Seems natural to me that emulate -L should extend the localisation
behaviour to traps as well as options. This also gives an automatic upgrade
if you move from 3.0.6 to 3.1.6 or 4 or whatever.
Also, dosetopt() is overkill for setting the options; it's really intended
for a generic option number where you need to test for side effects.
--- Doc/Zsh/builtins.yo.lt3 Mon Jun 21 09:28:19 1999
+++ Doc/Zsh/builtins.yo Tue Jun 22 17:38:17 1999
@@ -224,10 +224,11 @@
mode, except for certain options describing the interactive
environment; otherwise, only those options likely to cause portability
problems in scripts and functions are altered. If the tt(-L) option
-is given, the option tt(LOCAL_OPTIONS) will be set as well, causing
-the effect of the tt(emulate) command to be local to the immediately
-surrounding shell function, if any; normally this is turned off in all
-emulation modes except tt(ksh).
+is given, the options tt(LOCAL_OPTIONS) and tt(LOCAL_TRAPS) will be set as
+well, causing the effects of the tt(emulate) command and any tt(setopt) and
+tt(trap) commands to be local to the immediately surrounding shell
+function, if any; normally these options are turned off in all emulation
+modes except tt(ksh).
)
findex(enable)
cindex(enabling commands)
--- Src/builtin.c.lt3 Sun Jun 20 14:36:17 1999
+++ Src/builtin.c Tue Jun 22 17:36:18 1999
@@ -3074,7 +3074,7 @@
{
emulate(*argv, ops['R']);
if (ops['L'])
- dosetopt(LOCALOPTIONS, 1, 0);
+ opts[LOCALOPTIONS] = opts[LOCALTRAPS] = 1;
return 0;
}
--
Peter Stephenson <pws@ibmth.df.unipi.it> Tel: +39 050 844536
WWW: http://www.ifh.de/~pws/
Dipartimento di Fisica, Via Buonarroti 2, 56127 Pisa, Italy
reply other threads:[~1999-06-22 15:47 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=9906221547.AA25847@ibmth.df.unipi.it \
--to=pws@ibmth.df.unipi.it \
--cc=zsh-workers@sunsite.auc.dk \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
Code repositories for project(s) associated with this public inbox
https://git.vuxu.org/mirror/zsh/
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).