From: "Bart Schaefer" <schaefer@candle.brasslantern.com>
To: A.Main@dcs.warwick.ac.uk (Zefram), Zoltan Hidvegi <hzoli@cs.elte.hu>
Cc: segal@morgan.com, zsh-workers@math.gatech.edu (Zsh workers list)
Subject: Re: Bug in case stmt with '('
Date: Tue, 23 Jul 1996 09:25:57 -0700 [thread overview]
Message-ID: <960723092557.ZM2043@candle.brasslantern.com> (raw)
In-Reply-To: Zoltan Hidvegi <hzoli@cs.elte.hu> "Re: Bug in case stmt with '('" (Jul 23, 4:08pm)
On Jul 23, 4:08pm, Zoltan Hidvegi wrote:
} Subject: Re: Bug in case stmt with '('
}
} The best would be to handle it in glob.c but here it is a problem how can
} we distinguish ( foo\ | bar ) from ( foo | bar ).
The `if (incasepat && ...)' patch that I sent handles this correctly.
The only drawback I've found so far to my patch is that if you *don't*
use the POSIX balanced-parens syntax for "case", then parenthesized
patterns can't have *meaningful* spaces in them.
I don't think this is a significant difficulty, since "case" already
has the `|' syntax so there's not much reason to use glob-grouping
parens in "case" patterns in the first place.
} The simplest solution is to convert every unquoted space
} and TAB which is inside a globbing paren to a null-space and null-tab token
} and later in glob.c a null-space or null-tab is either treated as space/tab
} or discarded if it is adjacent to | or comes after a `(' of before a `)'
} (or it may be better to disard these after `)' and before `(' as well).
}
} This seems to be a quite simple solution.
Maybe, but not simpler than mine, and it's solving a problem that I don't
think we ought to be solving -- we don't need to be able to ignore spaces
in generalized glob patterns, only in "case" statements. I don't like the
suggestion to do it everywhere.
--
Bart Schaefer Brass Lantern Enterprises
http://www.well.com/user/barts http://www.nbn.com/people/lantern
New male in /home/schaefer:
>N 2 Justin William Schaefer Sat May 11 03:43 53/4040 "Happy Birthday"
next prev parent reply other threads:[~1996-07-23 16:29 UTC|newest]
Thread overview: 75+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <17651.199607222123@stone.dcs.warwick.ac.uk>
1996-07-23 14:08 ` Zoltan Hidvegi
1996-07-23 16:25 ` Bart Schaefer [this message]
2001-04-27 19:54 comptest* failed to load module: zsh/termcap Peter Whaite
2001-04-28 6:10 ` Bart Schaefer
2001-04-28 8:55 ` Andrej Borsenkow
2001-04-30 15:44 ` Peter Whaite
2001-04-30 16:30 ` Bart Schaefer
2001-04-30 16:47 ` Andrej Borsenkow
2001-04-30 17:16 ` PATCH: " Bart Schaefer
2001-04-30 18:54 ` Peter Whaite
2001-04-30 19:12 ` Bart Schaefer
2001-04-30 15:32 ` Peter Whaite
-- strict thread matches above, loose matches on Subject: below --
1997-01-31 10:47 history-search-backward Peter Stephenson
1997-01-31 12:16 ` history-search-backward Zefram
1997-01-31 12:42 ` history-search-backward Peter Stephenson
1997-01-31 15:02 ` history-search-backward Bart Schaefer
1997-01-31 15:23 ` history-search-backward Vinnie Shelton
1997-01-31 19:02 ` history-search-backward Zoltan Hidvegi
1997-01-31 19:13 ` history-search-backward Wayne Davison
1996-12-13 2:30 fifo configure check Zoltan Hidvegi
1996-10-31 14:41 Parameter expansion bug? Anthony Heading
1996-10-31 15:13 ` Hrvoje Niksic
1996-10-31 16:12 ` Zoltan Hidvegi
1996-10-31 20:49 ` Anthony Heading
[not found] <199607191600.SAA08613@bolyai.cs.elte.hu>
[not found] ` <9607211853.ZM979@morgan.com>
[not found] ` <960721233102.ZM22696@candle.brasslantern.com>
[not found] ` <schaefer>
1996-07-22 6:53 ` Bug in case stmt with '(' Bart Schaefer
1996-06-22 20:12 BUG: useheap in doexpandhist() Bart Schaefer
1996-06-02 17:59 Builtin append() and prepend() to PATH, CDPATH, etc Fung-Chai Lim
1996-06-02 21:26 ` Zoltan Hidvegi
[not found] ` <1062.199606041027@stone.dcs.warwick.ac.uk>
1996-04-10 10:31 History file locking? Russell Senior
1996-04-10 12:12 ` Zefram
1996-04-10 12:41 ` Zoltan Hidvegi
1996-04-10 12:50 ` Zefram
1996-04-10 21:21 ` Zoltan Hidvegi
[not found] ` <A.Main@dcs.warwick.ac.uk>
1996-04-10 16:26 ` Barton E. Schaefer
1996-04-10 17:08 ` Anthony Heading
[not found] ` <aheading@jpmorgan.com>
1996-04-10 17:12 ` Barton E. Schaefer
1996-04-10 18:03 ` Anthony Heading
1996-06-05 21:25 ` Builtin append() and prepend() to PATH, CDPATH, etc Bart Schaefer
[not found] ` <schaefer@candle.brasslantern.com>
1996-06-10 19:22 ` Clinton Bunch
1996-06-10 19:54 ` Bart Schaefer
1996-06-10 20:24 ` Hrvoje Niksic
1996-06-10 20:58 ` Clinton Bunch
1996-06-24 18:26 ` BUG: useheap in doexpandhist() Barton E. Schaefer
1996-06-24 19:11 ` Zoltan Hidvegi
1996-06-24 21:20 ` Barton E. Schaefer
1996-06-25 0:01 ` Zoltan Hidvegi
1996-06-25 0:22 ` Barton E. Schaefer
1996-07-23 20:01 ` Bug in case stmt with '(' Morris M. Siegel
1996-07-23 21:55 ` Bart Schaefer
1996-07-24 8:29 ` Zefram
1996-07-24 9:52 ` Peter Stephenson
1996-07-24 11:10 ` Bart Schaefer
1996-07-24 14:16 ` Zoltan Hidvegi
1996-03-12 19:03 Buffered stderr on Linux Zoltan Hidvegi
1996-03-12 19:24 ` Bart Schaefer
1996-03-12 20:26 ` Zoltan Hidvegi
1996-03-12 23:17 ` Zoltan Hidvegi
1996-03-13 0:16 ` Steven L Baur
[not found] ` <hzoli@cs.elte.hu>
1996-03-16 18:46 ` Bart Schaefer
1996-04-10 21:41 ` History file locking? Barton E. Schaefer
1996-04-10 22:16 ` Daniel Dignam
1996-06-26 13:51 ` Use of qualifiers without glob pattern? Bart Schaefer
1996-06-26 14:01 ` Regression tests Bas V. de Bakker
1996-06-26 14:52 ` Use of qualifiers without glob pattern? Zoltan Hidvegi
1996-06-26 15:54 ` Bart Schaefer
1996-06-27 1:03 ` Zoltan Hidvegi
1996-10-31 16:55 ` Parameter expansion bug? Bart Schaefer
1996-10-31 16:57 ` Hrvoje Niksic
1996-10-31 17:04 ` Zoltan Hidvegi
1996-10-31 17:47 ` Bart Schaefer
1996-12-13 3:37 ` fifo configure check Bart Schaefer
1996-12-13 8:58 ` fifo configure check (seems to work here: Solaris 2.4) C. v. Stuckrad
1996-12-13 14:44 ` fifo configure check Zefram
1996-12-13 17:33 ` Zoltan Hidvegi
[not found] ` <wayne@clari.net>
1997-01-31 22:01 ` history-search-backward Bart Schaefer
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=960723092557.ZM2043@candle.brasslantern.com \
--to=schaefer@candle.brasslantern.com \
--cc=A.Main@dcs.warwick.ac.uk \
--cc=hzoli@cs.elte.hu \
--cc=schaefer@nbn.com \
--cc=segal@morgan.com \
--cc=zsh-workers@math.gatech.edu \
/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).