From mboxrd@z Thu Jan 1 00:00:00 1970 Path: main.gmane.org!not-for-mail From: jasonr@btinternet.com Newsgroups: gmane.emacs.devel Subject: RE: [Patch for Windows implementation of play-sound] Date: Tue, 22 Oct 2002 10:28:34 +0100 (BST) Sender: emacs-devel-admin@gnu.org Message-ID: <7014725.1035278914934.JavaMail.root@127.0.0.1> NNTP-Posting-Host: main.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Transfer-Encoding: 7bit X-Trace: main.gmane.org 1035279037 8418 80.91.224.249 (22 Oct 2002 09:30:37 GMT) X-Complaints-To: usenet@main.gmane.org NNTP-Posting-Date: Tue, 22 Oct 2002 09:30:37 +0000 (UTC) Cc: emacs-devel@gnu.org Return-path: Original-Received: from quimby.gnus.org ([80.91.224.244]) by main.gmane.org with esmtp (Exim 3.35 #1 (Debian)) id 183vMy-0002BP-00 for ; Tue, 22 Oct 2002 11:30:36 +0200 Original-Received: from monty-python.gnu.org ([199.232.76.173]) by quimby.gnus.org with esmtp (Exim 3.12 #1 (Debian)) id 183vO6-00043X-00 for ; Tue, 22 Oct 2002 11:31:47 +0200 Original-Received: from localhost ([127.0.0.1] helo=monty-python.gnu.org) by monty-python.gnu.org with esmtp (Exim 4.10) id 183vLb-0004gW-00; Tue, 22 Oct 2002 05:29:11 -0400 Original-Received: from list by monty-python.gnu.org with tmda-scanned (Exim 4.10) id 183vLD-0004YG-00 for emacs-devel@gnu.org; Tue, 22 Oct 2002 05:28:47 -0400 Original-Received: from mail by monty-python.gnu.org with spam-scanned (Exim 4.10) id 183vLB-0004Xt-00 for emacs-devel@gnu.org; Tue, 22 Oct 2002 05:28:46 -0400 Original-Received: from actinium.btinternet.com ([194.73.73.66]) by monty-python.gnu.org with esmtp (Exim 4.10) id 183vLB-0004Xm-00 for emacs-devel@gnu.org; Tue, 22 Oct 2002 05:28:45 -0400 Original-Received: from fermium ([194.75.226.124]) by actinium.btinternet.com with esmtp (Exim 3.22 #8) id 183vL0-0005pf-00; Tue, 22 Oct 2002 10:28:34 +0100 Original-Received: from 193.192.75.200 by fermium ([194.75.226.124]); Tue, 22 Oct 02 10:28:34 BST Original-To: Bkey1@tampabay.rr.com, lektu@terra.es X-Mailer: talk21.com WAS v2 Errors-To: emacs-devel-admin@gnu.org X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.0.11 Precedence: bulk List-Help: List-Post: List-Subscribe: , List-Id: Emacs development discussions. List-Unsubscribe: , List-Archive: Xref: main.gmane.org gmane.emacs.devel:8646 X-Report-Spam: http://spam.gmane.org/gmane.emacs.devel:8646 In addition to finding another way around the WINVER problem, I think you should just define HAVE_SOUND instead of introducing a new conditional USE_W32SOUND. I am also concerned about the following line: + sprintf(sz_cmd_buf,"open \"%s\" alias SulliTech_PlaySound_Device wait",psz_file); What is the significance of SulliTech? It seems to me that is either specific to your hardware, or you have copied the code from somewhere, which could expose the FSF to legal risk if we install it. > > I've asked the author about it. Among other things, when compiling with > > MSVC it requires the newest release of the compiler and/or the SDK, and > > That's Not Good IMHO. > > This is not true. I did not say that the newest version of MSSDK is > required. I said that I have it installed. My patch will work with the > include files that are included with Visual C++ 6.0. You will, however, see > a large number of warning messages due to the following segment of the > version of WINDOWS.H that is included with Visual C++ 6.0: > > #if(WINVER >= 0x0500) > #pragma message ("") > #pragma message ("NOTE: WINVER has been defined as 0x0500 or greater which > enables") > #pragma message ("Windows NT 5.0 and Windows 98 features. When these > headers were released,") > #pragma message ("Windows NT 5.0 beta 1 and Windows 98 beta 2.1 were the > current versions.") > #pragma message ("") > #pragma message ("For this release when WINVER is defined as 0x0500 or > greater, you can only") > #pragma message ("build beta or test applications. To build a retail > application,") > #pragma message ("set WINVER to 0x0400 or visit > http://www.microsoft.com/msdn/sdk") > #pragma message ("to see if retail Windows NT 5.0 or Windows 98 headers > are available.") > #pragma message ("") > #pragma message ("See the SDK release notes for more information.") > #pragma message ("") > #endif > > > This warning message is displayed because I had to define WINVER and > _WIN32_WINNT to at least 0x0500 to avoid a compilation error I was > experiencing. I do not at this time have the details about that compilation > error.