[haiku-commits] Re: r33660 - haiku/trunk/src/apps/mediaplayer/media_node_framework/audio

  • From: David McPaul <dlmcpaul@xxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Tue, 20 Oct 2009 08:07:59 +1100

Stephen,

Why can't we use the Audio Mixer for this, it already does format
conversion and channel mapping?

2009/10/20  <superstippi@xxxxxx>:
> Author: stippi
> Date: 2009-10-19 20:24:11 +0200 (Mon, 19 Oct 2009)
> New Revision: 33660
> Changeset: http://dev.haiku-os.org/changeset/33660/haiku
>
> Modified:
>   
> haiku/trunk/src/apps/mediaplayer/media_node_framework/audio/AudioChannelConverter.cpp
> Log:
> Most simplistic support for 5 channels. Not actually tested. Uses the first
> two channels of five and ignores the rest.
>
>
> Modified: 
> haiku/trunk/src/apps/mediaplayer/media_node_framework/audio/AudioChannelConverter.cpp
> ===================================================================
> --- 
> haiku/trunk/src/apps/mediaplayer/media_node_framework/audio/AudioChannelConverter.cpp
>        2009-10-19 18:23:10 UTC (rev 33659)
> +++ 
> haiku/trunk/src/apps/mediaplayer/media_node_framework/audio/AudioChannelConverter.cpp
>        2009-10-19 18:24:11 UTC (rev 33660)
> @@ -61,6 +61,18 @@
>                                        break;
>                        }
>                        break;
> +               case 5:
> +                       switch (outChannels) {
> +                               case 2:
> +                                       for (int32 i = 0; i < frames; i++) {
> +                                               outBuffer[0] = inBuffer[0];
> +                                               outBuffer[1] = inBuffer[1];
> +                                               inBuffer += 5;
> +                                               outBuffer += 2;
> +                                       }
> +                                       break;
> +                       }
> +                       break;
>        }
>  }
>
>
>
>



-- 
Cheers
David

Other related posts: