Subject: Re: Removig generic optstr_get
To: Pavel Cahyna <pavel.cahyna@matfyz.cz>
From: Quentin Garnier <cube@cubidou.net>
List: tech-kern
Date: 10/25/2006 00:44:52
--utPK4TBebyzZxMrE
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Tue, Oct 24, 2006 at 10:20:49PM +0200, Pavel Cahyna wrote:
> On Tue, Oct 24, 2006 at 02:46:18PM +0200, Julio M. Merino Vidal wrote:
> > Hello,
> >=20
> > [ please CC me any replies ]
> >=20
> > When the Multiboot code was added, I also added a generic function
> > (optstr_get) used to parse "option strings" of the form: "a=3Db c=3Dd .=
..
> > x=3Dy"  The idea was to later use this interface in other places such as
> > the mount options passed during a mount(2).
> >=20
> > Since then, proplib was added which completely replaces this idea with
> > a more powerful framework (not to mention that the optstr_get
> > interface was probably suboptimal).  Therefore I'd like to do some
> > cleanup:
> > - Remove kern/subr_optstr.c and related manual pages and headers.
> > - Move optstr_get into arch/i386/i386/multiboot.c and make it static,
> >  as this is the only place where this functionality is used.  (And I gu=
ess
> >  it shouldn't be used anywhere else thanks to proplib.)
> >=20
> > Objections?
>=20
> Is there a real need for this change? I would prefer leaving things as
> they are now.
>=20
> (We might want to support richer boot parameters with other boot methods
> than Multiboot one day.)

And proplib will still be there for that.

--=20
Quentin Garnier - cube@cubidou.net - cube@NetBSD.org
"You could have made it, spitting out benchmarks
Owe it to yourself not to fail"
Amplifico, Spitting Out Benchmarks, Hometakes Vol. 2, 2005.

--utPK4TBebyzZxMrE
Content-Type: application/pgp-signature
Content-Disposition: inline

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.5 (NetBSD)

iQEVAwUBRT6XZNgoQloHrPnoAQLO+AgAsOkYqVXyYwmiq+UFwMYrjj2RwMmZ0wAq
+po2PAyQiP1cbG/ooz+DDoUEoof5dLc1BJUR0x+fmpV0Tms31CcvATbmOyH8IBjx
2+A7grMpGDH3E+TaTPXua2KF302qwOHX/p5LC+5MH6bkZMAbcOGwW5ed78/utm+e
BJ784A0G9DUgk/XyFg9QEY/ldfBvK/7j/JgwSUxzaTvStnsBr6MmpLb5P/pVNXgR
8DIavOyXPUfbpyeykoO2ImIOUNVMrViBBb4JBfCNqfAAx9Te3Slbvb8CM9IJlRxu
/ZKXnmrRR1DqljxnJYneH/d861zP4Q00xdKbNnjRFRvsGVbfK4K+BA==
=06BN
-----END PGP SIGNATURE-----

--utPK4TBebyzZxMrE--