Re: cvs commit: src/lib/libutil Makefile kld.3 kld.c libutil.h

2006-02-18 Thread Dag-Erling Smørgrav
Pawel Jakub Dawidek <[EMAIL PROTECTED]> writes: > static void > load_module(void) > { > > if (!kld_isloaded("g_%s", class_name) && > kld_load("geom_%s", class_name) == -1) { > err(1, "failed to load geom_%s module", class_name); > } > } you have to admit this is

Re: cvs commit: src/lib/libutil Makefile kld.3 kld.c libutil.h

2006-02-18 Thread Pawel Jakub Dawidek
On Sat, Feb 18, 2006 at 06:27:44PM +0100, Dag-Erling Sm?rgrav wrote: +> Pawel Jakub Dawidek <[EMAIL PROTECTED]> writes: +> > It doesn't change your API, just extends it. With this change I'll be +> > able to use this functions in geom(8) without wrappers around them. +> > And this is the whole poin

Re: cvs commit: src/lib/libutil Makefile kld.3 kld.c libutil.h

2006-02-18 Thread Dag-Erling Smørgrav
Pawel Jakub Dawidek <[EMAIL PROTECTED]> writes: > It doesn't change your API, just extends it. With this change I'll be > able to use this functions in geom(8) without wrappers around them. > And this is the whole point of having them, right? Please tell me what semantics you want instead of expec

Re: cvs commit: src/lib/libutil Makefile kld.3 kld.c libutil.h

2006-02-18 Thread Pawel Jakub Dawidek
On Sat, Feb 18, 2006 at 05:03:26PM +0100, Dag-Erling Sm?rgrav wrote: +> Pawel Jakub Dawidek <[EMAIL PROTECTED]> writes: +> > Sounds useful, thanks. It'll be even more useful if we can change it to: +> > +> >int kld_isloaded(const char *name, ...); +> >int kld_load(const char *name, ...); +>

Re: cvs commit: src/lib/libutil Makefile kld.3 kld.c libutil.h

2006-02-18 Thread Dag-Erling Smørgrav
Pawel Jakub Dawidek <[EMAIL PROTECTED]> writes: > Sounds useful, thanks. It'll be even more useful if we can change it to: > > int kld_isloaded(const char *name, ...); > int kld_load(const char *name, ...); > > What do you think? Not really. It just complicates the logic without addin

Re: cvs commit: src/lib/libutil Makefile kld.3 kld.c libutil.h

2006-02-18 Thread Pawel Jakub Dawidek
On Sat, Feb 18, 2006 at 11:25:28AM +, Dag-Erling Smorgrav wrote: +> des 2006-02-18 11:25:28 UTC +> +> FreeBSD src repository +> +> Modified files: +> lib/libutil Makefile libutil.h +> Added files: +> lib/libutil kld.3 kld.c +> Log: +> Add utility