All of lore.kernel.org
 help / color / mirror / Atom feed
From: Abramo Bagnara <abramo.bagnara@tin.it>
To: Jaroslav Kysela <perex@perex.cz>,
	alsa-devel <alsa-devel@lists.sourceforge.net>
Subject: Re: [alsa-cvslog] CVS: alsa-lib/src/pcm pcm.c,1.271,1.272
Date: Mon, 01 Mar 2004 22:17:32 +0100	[thread overview]
Message-ID: <4043A86C.4000406@tin.it> (raw)
In-Reply-To: <E1AxjBQ-00059u-Nx@sc8-pr-cvs1.sourceforge.net>

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?


-- 
Abramo Bagnara                       mailto:abramo.bagnara@tin.it

Opera Unica                          Phone: +39.0546.656023
Via Emilia Interna, 140
48014 Castel Bolognese (RA) - Italy


-------------------------------------------------------
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

       reply	other threads:[~2004-03-01 21:17 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <E1AxjBQ-00059u-Nx@sc8-pr-cvs1.sourceforge.net>
2004-03-01 21:17 ` Abramo Bagnara [this message]
2004-03-02  3:33   ` Re: [alsa-cvslog] CVS: alsa-lib/src/pcm pcm.c,1.271,1.272 Paul Davis
2004-03-02  7:28   ` Clemens Ladisch
2004-03-02  7:53   ` Jaroslav Kysela

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4043A86C.4000406@tin.it \
    --to=abramo.bagnara@tin.it \
    --cc=alsa-devel@lists.sourceforge.net \
    --cc=perex@perex.cz \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.