Jaroslav Kysela wrote:
>
> On Wed, 23 Oct 2002, Abramo Bagnara wrote:
>
> > Jaroslav Kysela wrote:
> > >
> > > Added ommited clear() functions
> > >
> > > Index: pcm.c
> > > /**
> > > + * \brief clear snd_pcm_hw_params_t structure
> > > + * \param obj pointer to structure
> > > + */
> > > +void snd_pcm_hw_params_clear(snd_pcm_hw_params_t *obj)
> > > +{
> > > + assert(obj);
> > > + memset(obj, 0, snd_pcm_hw_params_sizeof());
> > > +}
> >
> > Why that _clear functions?
> > What's the expected semantic for "clear"?
> >
> > When we designed new API I took in consideration them, but the lack of
> > usefulness and the ambiguity of semantic has pushed me to avoid them.
>
> Well, you're right that the name is misleading, but wouldn't be good to
> let to reuse a structure? Perhaps, _init suffix is better.
This is fine if we state in documentation that the object will go in the
same state than after OBJ_alloca/OBJ_malloc.
In this case I think that we have to change the memset in the OBJ_alloca
and OBJ_malloc function to OBJ_init call.
That apart there are some OBJs for which the _init function is nonsense.
snd_pcm_hw_params and snd_pcm_sw_params are among that.
The only sensible initializers for snd_pcm_hw_params are the existing
snd_pcm_hw_params_any and snd_pcm_hw_params_current.
Same for snd_pcm_sw_params_current.
To resume: I think that your proposal may have some reason for some OBJ
(but I'm not sure that in this specific ALSA context they exist), but I
think that to have it for OBJs with sane initializer is wrong.
--
Abramo Bagnara mailto:abramo.bagnara@;libero.it
Opera Unica Phone: +39.546.656023
Via Emilia Interna, 140
48014 Castel Bolognese (RA) - Italy
-------------------------------------------------------
This sf.net email is sponsored by: Influence the future
of Java(TM) technology. Join the Java Community
Process(SM) (JCP(SM)) program now.
http://ads.sourceforge.net/cgi-bin/redirect.pl?sunm0002en
_______________________________________________
Alsa-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/alsa-devel
