<br><br><div class="gmail_quote">On Sun, Feb 13, 2011 at 2:00 AM, Jean-Baptiste Kempf <span dir="ltr"><<a href="mailto:jb@videolan.org">jb@videolan.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
On Sun, Feb 13, 2011 at 01:48:00AM +0530, Pankaj yadav wrote :<br>
> #define bool int<br>
   What for?<br>
<br>
> #include <mda\common\audio.h><br>
> #include <Mda\Client\Utility.h><br>
> #include <MdaAudioOutputStream.h><br>
> #if !defined(__MDA_COMMON_RESOURCE_H__)<br>
> #include <Mda\Common\Resource.h><br>
> #endif<br>
Weird inclusions<br>
<br></blockquote><div>Why? <br>
</div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
> #define FRAME_SIZE 12442<br>
> #define BUFFERSIZE 12442<br>
Why this number?<br>
<br></blockquote><div>We need at least one of them..ryt? latter removed.. Done in attached</div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

> vlc_module_begin();<br>
>     set_description( _("Symbian audio output") );<br>
>     set_capability( "audio output", 40 );<br>
>     add_shortcut( "sdl" );<br>
>     set_callbacks( Open, Close );<br>
<br></blockquote><div>Added a lot of things.Done in attached :) <br>
</div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
missing a lot of things, see other modules... And sdl...<br>
<br>
>     if((desired.freq < 8000) || (desired.freq > 96000))<br>
>     {<br>
>         return VLC_EGENERIC;<br>
>     }<br>
>     if((desired.freq >= 8000) && (desired.freq < 11025))<br>
>     {<br>
>         SymbianSound.v_freq = 0x10;<br>
>     }<br>
>     if((desired.freq >= 11025) && (desired.freq < 12000))<br>
>     {<br>
>         SymbianSound.v_freq = 0x40;<br>
>     }<br>
>     if((desired.freq >= 12000) && (desired.freq < 16000))<br>
>     {<br>
>         SymbianSound.v_freq = 0x80;<br>
>     }<br>
>     if((desired.freq >= 16000) && (desired.freq < 22050))<br>
>     {<br>
>         SymbianSound.v_freq = 0x100;<br>
>     }<br>
>     if((desired.freq >= 22050) && (desired.freq < 24000))<br>
>     {<br>
>         SymbianSound.v_freq = 0x400;<br>
>     }<br>
>     if((desired.freq >= 24000) && (desired.freq < 32000))<br>
>     {<br>
>         SymbianSound.v_freq = 0x800;<br>
>     }<br>
>     if((desired.freq >= 32000) && (desired.freq < 44100))<br>
>     {<br>
>         SymbianSound.v_freq = 0x1000;<br>
>     }<br>
>     if((desired.freq >= 44100) && (desired.freq < 48000))<br>
>     {<br>
>         SymbianSound.v_freq = 0x4000;<br>
>     }<br>
>     if((desired.freq >= 48000) && (desired.freq<64000))<br>
>     {<br>
>         SymbianSound.v_freq = 0x10000;<br>
>     }<br>
>     if((desired.freq >= 64000) && (desired.freq<96000))<br>
>     {<br>
>         SymbianSound.v_freq = 0x40000;<br>
>     }<br>
>     if(desired.freq == 96000)<br>
>     {<br>
>         SymbianSound.v_freq=0x20000;<br>
>     }<br>
You know, you can use else...<br>
<br></blockquote><div>Done in attached <br>
</div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
>         if ( var_Type( p_aout, "audio-device" ) == 0 )<br>
>         {<br>
>             var_Create( p_aout, "audio-device",<br>
>                         VLC_VAR_INTEGER | VLC_VAR_HASCHOICE );<br>
>             text.psz_string = _("Audio device");<br>
>             var_Change( p_aout, "audio-device", VLC_VAR_SETTEXT, &text, NULL );<br>
I doubt this is how we do now.<br>
<br></blockquote><div>SDL aout uses the same thing. <br>
</div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
> static void Play( aout_instance_t * p_aout )<br>
> {<br>
> /*Yes It should be Empty*/<br>
Very weird.<br>
<br></blockquote><div>Ask this question from SDL aout authors..but yes this works<br>
</div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
> void *createdevice(void * arg)<br>
> {<br>
>     if (!CActiveScheduler::Current())<br>
>     {<br>
>         SymbianSound.scheduler = new (ELeave) CActiveScheduler;<br>
>         if(SymbianSound.scheduler)<br>
>         {<br>
>         CActiveScheduler::Install(SymbianSound.scheduler);<br>
indentation is wrong.<br>
<br></blockquote><div>Done in attached <br>
</div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Comments are missing. This needs some polish, I would say.<br>
 <br>
</blockquote><div>Comments?<br>
</div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<br>
Best Regards,<br>
<font color="#888888"><br>
--<br>
Jean-Baptiste Kempf<br>
<a href="http://www.jbkempf.com/" target="_blank">http://www.jbkempf.com/</a><br>
+33 672 704 734<br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="http://mailman.videolan.org/listinfo/vlc-devel" target="_blank">http://mailman.videolan.org/listinfo/vlc-devel</a><br>
</font></blockquote></div><br><br clear="all"><br>-- <br>Pankaj Yadav<br><a href="http://people.videolan.org/%7Epk" target="_blank">http://people.videolan.org/~pk</a><br>Open Source Enthusiast<br>