Subject: Re: CVS commit: src/include
To: None <elad@netbsd.org>
From: YAMAMOTO Takashi <yamt@mwd.biglobe.ne.jp>
List: source-changes
Date: 02/18/2006 20:23:21
> Module Name:	src
> Committed By:	elad
> Date:		Sat Feb 18 10:53:33 UTC 2006
> 
> Modified Files:
> 	src/include: util.h
> 
> Log Message:
> Add the struct pw_policy forgotten in previous commit.

i don't think it's a good idea to expose this internal detail to callers.

how about:
	struct pw_policy; /* opaque */

	struct pw_policy *pw_policy_load(...);
	int pw_policy_test(const struct pw_policy *, ...);
	void pw_policy_free(struct pw_policy *);

btw, i think "pw" and "key" should be const.

YAMAMOTO Takashi