
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 TRANSITIONAL//EN">

<HTML>

<HEAD>

  <META HTTP-EQUIV="Content-Type" CONTENT="text/html; CHARSET=UTF-8">

  <META NAME="GENERATOR" CONTENT="GtkHTML/3.14.1">

</HEAD>

<BODY>

Hi,<BR>

<BR>

For the first comment, please read my other email regarding jack-fixes.patch.<BR>

<BR>

Here is a new version of the vorbis only patch.  I removed all my debuging stuff.  There was a return which should not have been commented -- solved.  As for the "aout_DateSet( &p_sys->end_date, 0 );" I deleted, it was a duplication, aout_DateInit already do this.<BR>

<BR>

<BR>

Le vendredi 31 août 2007 à 10:26 +0300, Rémi Denis-Courmont a écrit :

<BLOCKQUOTE TYPE=CITE>

<PRE>

<FONT COLOR="#000000">Le lundi 27 août 2007, Pierre-Luc Beaudoin a écrit :</FONT>

<FONT COLOR="#000000">> Here is my patch introducing :</FONT>

<FONT COLOR="#000000">> - a Jack input UI (qt4 only) under Capture Device tab</FONT>

<FONT COLOR="#000000">> - corrections to bugs in sound quality with Jack as an Input</FONT>

<FONT COLOR="#000000">> - allow to stream up to 8 channels of sound through Vorbis.</FONT>


<FONT COLOR="#000000">If I understand correctly, you are adding a dependency on libm and float </FONT>

<FONT COLOR="#000000">point numbers just for the sake of rounding a value to a power of 2. I </FONT>

<FONT COLOR="#000000">doubt that is really wise.</FONT>


<FONT COLOR="#000000">Also, some of the commenting out in the ogg patch seems bogus.</FONT>


<FONT COLOR="#000000">Regards,</FONT>


</PRE>

</BLOCKQUOTE>

<TABLE CELLSPACING="0" CELLPADDING="0" WIDTH="100%">

<TR>

<TD>

--<BR>

<B>Pierre-Luc Beaudoin</B><BR>

Savoir-faire Linux<BR>

<BR>

<BR>

</TD>

</TR>

</TABLE>

</BODY>

</HTML>