This is now checked into 310Atlas, 3_4_10Atlas. As 347Atlas doesn't contain PulseAudio support, only relevant audlinux_alsa.cpp change is checked into 347Atlas, as mentioned in the CR, to make all branches consistent.
-- Daniel Yek. Daniel Yek wrote: > I'll be checking this in later this afternoon if there is no comment. > > Modified by: [email protected] > Date: 12/4/2009 > Project: RealPlayer for Netbook > > Synopsis: > Fixed PulseAudio PlaybackMultiChannelAsStereo support & refine ALSA's > implementation. > > Overview: > PlaybackMultiChannelAsStereo support code belongs in the: > CAudioOutLinux*::_CheckFormat( const HXAudioFormat* pFormat ) > method. > > On ALSA, _CheckFormat() also calls _OpenAudio(). > As a result, the PlaybackMultiChannelAsStereo support code > was accidentally implemented in _OpenAudio() instead, > which worked for ALSA, > but when the implementation was copied to PulseAudio, > it didn't work as expected. > > This change fixed PulseAudio's PlaybackMultiChannelAsStereo support and > refined ALSA's implementation by moving it to _CheckFormat() > to avoid further confusion. > > > Files Modified: > audio/device/platform/unix/audlinux_alsa.cpp > audio/device/platform/unix/audlinux_pulseaudio.cpp > > Image Size and Heap Use impact (Client -Only): > None. > > Platforms and Profiles Affected: > Linux > > Distribution Libraries Affected: > None. > > Distribution library impact and planned action: > None. > > Platforms and Profiles Build Verified: > Profile: helix_client_moblin > Platform: linux-2.2-libc6-gcc32-i586 > > Platforms and Profiles Functionality verified: > Profile: helix_client_moblin > Platform: linux-2.2-libc6-gcc32-i586 > > Branch: 310Atlas, 3_4_10Atlas. > Also 347Atlas for the ALSA change. > > Copyright assignment: I am a RealNetworks employee. > _______________________________________________ Audio-dev mailing list [email protected] http://lists.helixcommunity.org/mailman/listinfo/audio-dev
