Re: [PATCH/RFC] resolver_r.3: New page documenting res_ninit() et al

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

IIRC bind 8 (or libbind) also comes with its own manpages. Ages ago I
stole the info from their resolver.3 page and merged it with the page
distributed with glibc (the current version is taken from there as
well, IIRC). The patch got somehow lost through the cracks, I think...

Or do you see any licensing issues with doing that?

OTOH, if you want to keep your approach and add another page for the
reentrant functions, you might want to add a pointer in resolver.3 so
people know there's another page as well :).

Just my 2 cents,
Stefan.

On Mon, Jul 4, 2011 at 8:33 AM, Jonathan Nieder <jrnieder@xxxxxxxxx> wrote:
> Hi again,
>
> Jonathan Nieder wrote:
>
>> --- /dev/null
>> +++ b/man3/resolver_r.3
>> @@ -0,0 +1,136 @@
> [...]
>> +.B typedef struct state *res_state;
>
> Looking over [1], it occurs to me that treating "res_state" as an
> opaque type doesn't work if this API is to be actually usable.  A
> typical program will look like this:
>
>        struct __res_state res;
>
>        if (res_ninit(&res))
>                die_errno("cannot initialize resolver");
>        ...
>
> Is "struct __res_state" really supposed to be part of the public
> API?  I guess it is.
>
> So I'd suggest squashing in the mini-patch below to reflect that.
> Sorry for the noise.
>
> [1] https://lists.isc.org/pipermail/bind-users/2008-July/070847.html
>
> Signed-off-by: Jonathan Nieder <jrnieder@xxxxxxxxx>
> ---
>  man3/resolver_r.3 |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/man3/resolver_r.3 b/man3/resolver_r.3
> index f2ced438..a70834b4 100644
> --- a/man3/resolver_r.3
> +++ b/man3/resolver_r.3
> @@ -13,7 +13,8 @@ res_nsend \- reentrant resolver routines
>  .nf
>  .B #include <resolv.h>
>  .sp
> -.B typedef struct state *res_state;
> +.B struct __res_state;
> +.B typedef struct __res_state *res_state;
>  .sp
>  .BI "int res_ninit(res_state " statep ");"
>  .sp
> --
> 1.7.6
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-man" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
--
To unsubscribe from this list: send the line "unsubscribe linux-man" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Kernel Documentation]     [Netdev]     [Linux Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux