* Re: Re: [alsa-cvslog] CVS: alsa-lib/src/pcm pcm.c,1.271,1.272
2004-03-01 21:17 ` [alsa-cvslog] CVS: alsa-lib/src/pcm pcm.c,1.271,1.272 Abramo Bagnara
@ 2004-03-02 3:33 ` Paul Davis
2004-03-02 7:28 ` Clemens Ladisch
2004-03-02 7:53 ` Jaroslav Kysela
2 siblings, 0 replies; 4+ messages in thread
From: Paul Davis @ 2004-03-02 3:33 UTC (permalink / raw)
To: Abramo Bagnara; +Cc: Jaroslav Kysela, alsa-devel
>Taken for granted that configuration space on entry of function is not
>empty ....
can this be taken for granted?
-------------------------------------------------------
SF.Net is sponsored by: Speed Start Your Linux Apps Now.
Build and deploy apps & Web services for Linux with
a free DVD software kit from IBM. Click Now!
http://ads.osdn.com/?ad_id=1356&alloc_id=3438&op=click
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: Re: [alsa-cvslog] CVS: alsa-lib/src/pcm pcm.c,1.271,1.272
2004-03-01 21:17 ` [alsa-cvslog] CVS: alsa-lib/src/pcm pcm.c,1.271,1.272 Abramo Bagnara
2004-03-02 3:33 ` Paul Davis
@ 2004-03-02 7:28 ` Clemens Ladisch
2004-03-02 7:53 ` Jaroslav Kysela
2 siblings, 0 replies; 4+ messages in thread
From: Clemens Ladisch @ 2004-03-02 7:28 UTC (permalink / raw)
To: Abramo Bagnara; +Cc: Jaroslav Kysela, alsa-devel
Abramo Bagnara wrote:
> Jaroslav Kysela ha scritto:
> > Log Message:
> > Fixed return comment for *near() functions
> >
> > + * \return 0 otherwise a negative error code if configuration space would become empty
>
> *near() functions semantic cannot return error by design.
>
> Taken for granted that configuration space on entry of function is not
> empty how it can become empty if I ask for *nearest* value?
>
> Obviously at least one configuration exists, then in the worst case such
> value is returned.
It may be possible that other errors than an empty configuration space
could occur. For example, future ALSA versions might want to fail if
an hotplug device has been unpluggeed, or the configuration space
could have been reduced due to some outside influence.
I don't think we can guarantee that this function never ever fails.
Maybe we should simply drop the "if configuration space would become
empty".
Regards,
Clemens
-------------------------------------------------------
SF.Net is sponsored by: Speed Start Your Linux Apps Now.
Build and deploy apps & Web services for Linux with
a free DVD software kit from IBM. Click Now!
http://ads.osdn.com/?ad_id=1356&alloc_id=3438&op=click
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [alsa-cvslog] CVS: alsa-lib/src/pcm pcm.c,1.271,1.272
2004-03-01 21:17 ` [alsa-cvslog] CVS: alsa-lib/src/pcm pcm.c,1.271,1.272 Abramo Bagnara
2004-03-02 3:33 ` Paul Davis
2004-03-02 7:28 ` Clemens Ladisch
@ 2004-03-02 7:53 ` Jaroslav Kysela
2 siblings, 0 replies; 4+ messages in thread
From: Jaroslav Kysela @ 2004-03-02 7:53 UTC (permalink / raw)
To: Abramo Bagnara; +Cc: alsa-devel
On Mon, 1 Mar 2004, Abramo Bagnara wrote:
> Jaroslav Kysela ha scritto:
> > Update of /cvsroot/alsa/alsa-lib/src/pcm
> > In directory sc8-pr-cvs1.sourceforge.net:/tmp/cvs-serv19814
> >
> > Modified Files:
> > pcm.c
> > Log Message:
> > Fixed return comment for *near() functions
> >
> > Index: pcm.c
> > ===================================================================
> > RCS file: /cvsroot/alsa/alsa-lib/src/pcm/pcm.c,v
> > retrieving revision 1.271
> > retrieving revision 1.272
> > diff -u -r1.271 -r1.272
> > --- pcm.c 23 Feb 2004 10:34:17 -0000 1.271
> > +++ pcm.c 1 Mar 2004 08:53:50 -0000 1.272
> > @@ -3418,7 +3418,7 @@
> > * \param pcm PCM handle
> > * \param params Configuration space
> > * \param val target channels count, returned chosen channels count
> > - * \return chosen channels count
> > + * \return 0 otherwise a negative error code if configuration space would become empty
>
> *near() functions semantic cannot return error by design.
>
> Taken for granted that configuration space on entry of function is not
> empty how it can become empty if I ask for *nearest* value?
>
> Obviously at least one configuration exists, then in the worst case such
> value is returned.
>
> Do you agree?
Ok, I replace the phrase with "if configuration space is empty".
Jaroslav
-----
Jaroslav Kysela <perex@suse.cz>
Linux Kernel Sound Maintainer
ALSA Project, SuSE Labs
-------------------------------------------------------
SF.Net is sponsored by: Speed Start Your Linux Apps Now.
Build and deploy apps & Web services for Linux with
a free DVD software kit from IBM. Click Now!
http://ads.osdn.com/?ad_id=1356&alloc_id=3438&op=click
^ permalink raw reply [flat|nested] 4+ messages in thread