I was checking out some VLC message logs and I noticed following when running with Waveout:<div>main debug: thread (waveOut Notification Thread) created at priority 1 (waveout.c:348)</div><div><br></div><div>when run with DirectX audio output:</div>
<div>main debug: thread (DirectSound Notification Thread) created at priority 15 (directx.c:321)</div><div><br></div><div>The priority is different. But according to source code</div><div><a href="http://git.videolan.org/?p=vlc.git;a=blob;f=modules/audio_output/waveout.c">http://git.videolan.org/?p=vlc.git;a=blob;f=modules/audio_output/waveout.c</a></div>
<div><br></div><div>Waveout uses VLC_THREAD_PRIORITY_OUTPUT which is defined in </div><div><a href="http://git.videolan.org/?p=vlc.git;a=blob;f=include/vlc_threads.h">http://git.videolan.org/?p=vlc.git;a=blob;f=include/vlc_threads.h</a></div>
<div><br></div><div><div>#define VLC_THREAD_PRIORITY_OUTPUT \</div><div>THREAD_PRIORITY_ABOVE_NORMAL</div><div><br></div><div>and that according to MSDN it shouldn't return 1 in any case.</div><div><a href="http://msdn.microsoft.com/en-us/library/ms685100(VS.85).aspx">http://msdn.microsoft.com/en-us/library/ms685100(VS.85).aspx</a></div>
<div><br></div><div>In VLC 0.8.6 the Waveout used VLC_THREAD_PRIORITY_HIGHEST but I can't find commit log that explains why it was changed.</div><div><br></div><div>To my knowledge 1 is way too small value priority for audio thread. I can send patch to change this, but I would like to know if there is some reason for this.</div>
<div><br></div><div><br></div><div>And BTW, I will merge all VLC related mailing to this email address. So I won't use agent 007 at luukku and firstname.surname at tut addresses anymore.</div></div>