regex.c change between 1.64 and 1.65

[Andreas Schwab]

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