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 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


[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