From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 6916 invoked by alias); 2 May 2012 14:26: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: 30465 Received: (qmail 17984 invoked from network); 2 May 2012 14:26:05 -0000 X-Spam-Checker-Version: SpamAssassin 3.3.2 (2011-06-06) on f.primenet.com.au X-Spam-Level: X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,RCVD_IN_DNSWL_NONE autolearn=ham version=3.3.2 Received-SPF: none (ns1.primenet.com.au: domain at closedmail.com does not designate permitted sender hosts) From: Bart Schaefer Message-id: <120502072525.ZM12766@torch.brasslantern.com> Date: Wed, 02 May 2012 07:25:25 -0700 In-reply-to: Comments: In reply to Mikael Magnusson "Re: PATCH: Fix some more missing locals for $match" (May 2, 2:51pm) References: <1335954913-3709-1-git-send-email-mikachu@gmail.com> <1335955692-4359-1-git-send-email-mikachu@gmail.com> <20120502115650.6c519661@pwslap01u.europe.root.pri> X-Mailer: OpenZMail Classic (0.9.2 24April2005) To: zsh-workers@zsh.org Subject: Re: PATCH: Fix some more missing locals for $match MIME-version: 1.0 Content-type: text/plain; charset=us-ascii On May 2, 2:51pm, Mikael Magnusson wrote: } } On 02/05/2012, Peter Stephenson wrote: } > Ideally, these should always be } > } > local -a match mbegin mend } } Ah, forgot about those, will include in the commit. I guess I can as } well check that all places that local match also local those. And } tabcompletion just reminded me of MBEGIN/MATCH/MEND too, but that's } probably my rc. While on the subject, match/MATCH mbegin/MBEGIN mend/MEND are not mentioned in the "Paramters Used by the Shell" documentation, nor are they vindex()ed anywhere in the doc *.yo files.