ports/156171: port multimedia/mplayer patch-libao2-ao_oss.c is incorrect

[Zhihao Yuan]

>Number:         156171
>Category:       ports
>Synopsis:       port multimedia/mplayer patch-libao2-ao_oss.c is incorrect
>Confidential:   no
>Severity:       non-critical
>Priority:       medium
>Responsible:    freebsd-ports-bugs
>State:          open
>Quarter:        
>Keywords:       
>Date-Required:
>Class:          change-request
>Submitter-Id:   current-users
>Arrival-Date:   Mon Apr 04 08:50:03 UTC 2011
>Closed-Date:
>Last-Modified:
>Originator:     Zhihao Yuan
>Release:        FreeBSD 8.2-STABLE amd64
>Organization:
Northern Illinois University
>Environment:
System: FreeBSD compaq.yuetime 8.2-STABLE FreeBSD 8.2-STABLE #0: Mon Mar 14 02:51:28 CDT 2011 root at compaq.yuetime:/usr/obj/usr/src/sys/HOUKAGO amd64


	
>Description:
	
	This patch moves the samplerate setting clause out of a condition block, which intends to make sure the samplerate is always set. But actually, there is another condition block at line 452, which makes these two blocks mutual exclusive. So, this patch is no needed.

	But! The author has reason to write the conditions like this! According to 4Front's document,
	http://www.4front-tech.com/pguide/audio.html
	the SNDCTLs must be set in this order: format, channels, sampling rate. So, this patch is incorrect.

>How-To-Repeat:
	
>Fix:

	
	Since the patch is unneeded and incorrect, please remove it.


>Release-Note:
>Audit-Trail:
>Unformatted: