[vlc-devel] Re: vlc: svn commit r13322 (gbazin)

[Gildas Bazin]

On Tuesday 22 November 2005 20:26, Subversion daemon wrote:
> r13322 | gbazin | 2005-11-22 21:26:31 +0100 (Tue, 22 Nov 2005) | 1 line
> Changed paths:
>    M /trunk/modules/audio_output/directx.c
> 
> * modules/audio_output/directx.c: fixed the way we get the current latency 
in the driver. fixes the continuous resampling issues with s16l and might 
also fix the fl32 problems. This needs some heavy testing.
> 

Ok, turns out we still have problems.
While s16l seems to work correctly if nothing else is using the soundcard at 
the same time, it is completely falling over if something is (even mouse 
click sounds upset our audio output).
The fl32 issue with nforce drivers also hasn't been solved.

Our directx audio output module uses a feature of directx called position 
notification. Basically directx will trigger notification events when the 
audio playback reaches predefined positions in the circular buffer.

Well, turns out that this feature is completely buggy. Even Microsoft 
apparently doesn't recommend using it because of bugs in both directx and 
audio drivers (of course this information was hidden somewhere and I only 
stumbled on it while investigating our problems).

That basically means that our directx output module needs to be rewritten 
using a polling mechanism instead of depending on this notification 
feature. This won't actually require too much work, only 3 or 4 hours of 
coding, but I would advise holding of the 0.8.4 release until this is done 
and has received a minimum amount of testing. Hopefully I'll get around to 
doing it tomorrow evening.

--
Gildas

-- 
This is the vlc-devel mailing-list, see http://www.videolan.org/vlc/
To unsubscribe, please read http://developers.videolan.org/lists.html