CVS log for src/lib/libc/regex/regerror.c

Up to [cvs.NetBSD.org] / src / lib / libc / regex

Request diff between arbitrary revisions

Keyword substitution: kv
Default branch: MAIN

regerror(3): Allow null errbuf if errbuf_size is zero.

The man page says:

   If errbuf_size is 0, errbuf is ignored but the return value is still
   correct.

POSIX says:

   If errbuf_size is 0, regerror() shall ignore the errbuf argument,
   and return the size of the buffer needed to hold the generated
   string.

   https://pubs.opengroup.org/onlinepubs/9699919799/functions/regerror.html

from enh@google.com

Arrange for tools build

sync with FreeBSD:
    - NLS support
    - GNU extensions
    - bug fixes

Rearrange headers for better portability.

ANSIfy.

Nuke __P.

constify and remove __UNCONST.

WARNS=4

Move UCB-licensed code from 4-clause to 3-clause licence.

Patches provided by Joel Baker in PR 22280, verified by myself.

Sync with HEAD.

use strlcpy where it is more appropriate.

Delint.
Remove trailing ; from uses of __weak_alias().  The macro inserts this if
needed.

back out the #ifdef _DIAGNOSTIC argument checks; too many people complained.
_DIAGASSERT() is still retained.

The previous assertion change was incorrect: the preg argument is permitted to
be NULL (part of the interface definition!) unless errcode is REG_ATOI, which
is a non-portable extension.

Also, remove assertions from the local regatoi() function, as they are either
redundant or will never fail due to the buffer allocation for that argument in
regerror().

* use _DIAGASSERT() to check pointer arguments against NULL and file
  descriptors against -1 (as appropriate).
* add actual checks which to detect stuff that would trigger_DIAGASSERT(),
  and attempt to return a sane error condition.
* knf some code
* remove some `register' decls.

the first two items result in the addition of code similar to the
following in various functions:

		_DIAGASSERT(path != NULL)
	#ifdef _DIAGNOSTIC
		if (path == NULL) {
			errno = EFAULT;
			return (-1);
		}
	#endif

correct a comment, minor formatting

Delint:
	- we don't need to use longs; convert to use int32_t.
	- remove break statements after returns.
	- cast things properly.

const poisoning.

constify some tables.

remove obsolete register declarations

If port provides __weak_alias(), provide an Standard C and POSIX pure
identifier namespace by renaming non standard functions and variables
such that they have a leading underscore.  The library will use those
names internally.  Weak aliases are used to provide the original names
to the API.

This is only the first part of this change.  It is most of the functions
which are implemented in C for all NetBSD ports.  Subsequent changes are
to add the same support to the remaining C files, to assembly files, and
to the automagically generated assembly source used for system calls.
When all of the above is done, ports with weak alias support should add
a definition for __weak_alias to <sys/cdefs.h>.

Add braces in initializers.
Fix RCSID's
Remove unused variables

- convert unsafe strcpy(), strcat() and sprintf() to the `n' versions.
- some KNF.

add unintrusive Ids

clean up import; kill old cruft.

regex code from lite.  same as we're using, but slightly cleaner.

Update to Henry's Nov 2, 1993 alpha3.2 release.

Henry Spencer's POSIX.2 compatibile regular expression matching routines.