This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Ping: [PATCH] Add inline bsearch expansion


> 	* bits/stdlib-bsearch.h: New file.
> 	* stdlib/bsearch.c: Include bits/stdlib-bsearch.h.
> 	* stdlib/stdlib.h(bsearch): Add inline bsearch.

Space before that paren.

> --- /dev/null
> +++ b/bits/stdlib-bsearch.h
> @@ -0,0 +1,28 @@
> +#ifdef __USE_EXTERN_INLINES

Every file needs the standard header comment (descriptive first line, then
the standard copyright text).  Since the contents are copied from
bsearch.c, use the same copyright year range that's already in that file.

Don't put that #ifdef inside this file.
Instead, conditionalize its inclusion in stdlib.h.

> diff --git a/stdlib/bsearch.c b/stdlib/bsearch.c
> index 55b4f37..04a9d9d 100644
> --- a/stdlib/bsearch.c
> +++ b/stdlib/bsearch.c
> @@ -1,4 +1,4 @@
> -/* Copyright (C) 1991-2013 Free Software Foundation, Inc.
> +/* Copyright (C) 2013 Free Software Foundation, Inc.

Never remove old years from an existing copyright notice like this.


Thanks,
Roland


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