<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
  <meta content="text/html;charset=UTF-8" http-equiv="Content-Type">
</head>
<body bgcolor="#ffffff" text="#000000">
Hi,<br>
<br>
I've re written my patch with your suggestion so it doesn't need the
no_channels variable and assigns the ss.channels initially.<br>
The code looks a lot neater and the diff is attached.<br>
<br>
kind regards,<br>
Danny<br>
<br>
Rémi Denis-Courmont wrote:
<blockquote cite="mid:200812021914.41132.rem@videolan.org" type="cite">
  <pre wrap="">Le lundi 1 décembre 2008 13:16:21 Danny Wood, vous avez écrit :
  </pre>
  <blockquote type="cite">
    <pre wrap="">I have created a patch (against VLC-0.9.4) to the pulse audio output
plug-in for VLC to allow 5.1, 4.0, and stereo/mono streams to be
outputted to pulse to allow full surround or up-mixing based on the
input streams no. of channels.
The diff/patch is attached.
    </pre>
  </blockquote>
  <pre wrap=""><!---->
You can probably set ss.channels to no_channels instead of replicating the 
assignment for every case. Looks good otherwise.

Regards,

  </pre>
</blockquote>
</body>
</html>