This is the mail archive of the libc-hacker@sourceware.cygnus.com mailing list for the glibc project.

Note that libc-hacker is a closed list. You may look at the archives of this list, but subscription and posting are not open.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]

Re: regex.c change between 1.64 and 1.65

To: drepper at cygnus dot com (Ulrich Drepper)
Subject: Re: regex.c change between 1.64 and 1.65
From: Andreas Schwab <schwab at suse dot de>
Date: Tue, 8 Feb 2000 18:02:30 +0100
Cc: libc-hacker at sourceware dot cygnus dot com
References: <200002011145.MAA17263@hawking.suse.de> <m3g0v4fpev.fsf@localhost.localnet>

Ulrich Drepper <drepper@redhat.com> writes:

|> Andreas Schwab <schwab@suse.de> writes:
|> 
|> > What is this change in posix/regex.c supposed to fix?
|> 
|> As you can read in the ChangeLog, it's supposed to fix error value
|> settings.  I haven't investigated all places I changed in detail but
|> the changes should all be correct.

SUS2 says that malformed brace groups are undefined behaviour.  Previously
the regexp compiler took the brace as a literal instead, now it will
reject the regexp.  I'd prefer to restore the previous behaviour, since it
is compatible with other regexp matchers.

|> In this case it would mean that the `if' isn't necessary.

IMHO this is not the right way to fix the problem.

Andreas.

-- 
Andreas Schwab                                  "And now for something
SuSE Labs                                        completely different."
schwab@suse.de
SuSE GmbH, Schanzäckerstr. 10, D-90443 Nürnberg

References:
- regex.c change between 1.64 and 1.65
  - From: Andreas Schwab
- Re: regex.c change between 1.64 and 1.65
  - From: Ulrich Drepper

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]