[prev in list] [next in list] [prev in thread] [next in thread] 

List:       gstreamer-cvs
Subject:    Re: [0.11] gst-plugins-base: audio: add function to build audio format
From:       Stefan Kost <ensonic () hora-obscura ! de>
Date:       2011-08-19 22:52:16
Message-ID: 4E4EE920.1020003 () hora-obscura ! de
[Download RAW message or body]

On 08/19/11 16:02, Wim Taymans wrote:
> Module: gst-plugins-base
> Branch: 0.11
> Commit: 7db6fa37b49858e517230181605961b7b04face1
> URL:    http://cgit.freedesktop.org/gstreamer/gst-plugins-base/commit/?id=7db6fa37b49858e517230181605961b7b04face1
>  
> Author: Wim Taymans <wim.taymans@collabora.co.uk>
> Date:   Fri Aug 19 16:00:33 2011 +0200
> 
> audio: add function to build audio format
> 
> ---
> 
> gst-libs/gst/audio/audio.c |   42 ++++++++++++++++++++++++++++++++++++++++++
> gst-libs/gst/audio/audio.h |   14 +++++++++-----
> 2 files changed, 51 insertions(+), 5 deletions(-)
> 
> diff --git a/gst-libs/gst/audio/audio.c b/gst-libs/gst/audio/audio.c
> index ea415c3..b04b9e4 100644
> --- a/gst-libs/gst/audio/audio.c
> +++ b/gst-libs/gst/audio/audio.c
> @@ -102,6 +102,48 @@ static GstAudioFormatInfo formats[] = {
> };
> 
> /**
> + * gst_audio_format_build_int:
> + * @sign: signed or unsigned format
> + * @endianness: G_LITTLE_ENDIAN or G_BIG_ENDIAN
> + * @width: amount of bits used per sample
> + * @depth: amount of used bits in @width
> + *
> + * Construct a #GstAudioFormat with given parameters.
> + *
> + * Returns: a #GstAudioFormat or GST_AUDIO_FORMAT_UNKNOWN when no audio format
> + * exists with the given parameters.
> + */
would gst_audio_format_get_{integer,float}_id() or
gst_audio_format_get_{integer,float}_format() be a better name? Spelling
'int' out would make it less ambigous (int=internal?) and avoids us even
considering 'flt' for float.

Stefan

_______________________________________________
gstreamer-commits mailing list
gstreamer-commits@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/gstreamer-commits


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic