Alexandre Julliard : winejack.drv: Remove unnecessary ifdefs.

Alexandre Julliard julliard at winehq.org
Tue Mar 29 11:43:01 CDT 2011


Module: wine
Branch: master
Commit: d25763d4b6ad10713a33700dead3a8ff24d3380b
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=d25763d4b6ad10713a33700dead3a8ff24d3380b

Author: Alexandre Julliard <julliard at winehq.org>
Date:   Mon Mar 28 22:14:00 2011 +0200

winejack.drv: Remove unnecessary ifdefs.

---

 dlls/winejack.drv/Makefile.in |    3 +-
 dlls/winejack.drv/audio.c     |   72 ++++++++++++++++--------
 dlls/winejack.drv/jack.c      |  122 -----------------------------------------
 dlls/winejack.drv/jack.h      |   25 --------
 4 files changed, 49 insertions(+), 173 deletions(-)

diff --git a/dlls/winejack.drv/Makefile.in b/dlls/winejack.drv/Makefile.in
index 6a074e9..e7263d3 100644
--- a/dlls/winejack.drv/Makefile.in
+++ b/dlls/winejack.drv/Makefile.in
@@ -2,7 +2,6 @@ MODULE    = winejack.drv
 IMPORTS   = dxguid uuid winmm user32
 
 C_SRCS = \
-	audio.c \
-	jack.c
+	audio.c
 
 @MAKE_DLL_RULES@
diff --git a/dlls/winejack.drv/audio.c b/dlls/winejack.drv/audio.c
index da04824..ac8a0c0 100644
--- a/dlls/winejack.drv/audio.c
+++ b/dlls/winejack.drv/audio.c
@@ -51,7 +51,6 @@
 #include "mmddk.h"
 #include "dsound.h"
 #include "dsdriver.h"
-#include "jack.h"
 #include "wine/unicode.h"
 #include "wine/library.h"
 #include "wine/debug.h"
@@ -63,8 +62,6 @@
 
 WINE_DEFAULT_DEBUG_CHANNEL(wave);
 
-#ifdef SONAME_LIBJACK
-
 #define MAKE_FUNCPTR(f) static typeof(f) * fp_##f = NULL;
 
 /* Function pointers for dynamic loading of libjack */
@@ -86,6 +83,8 @@ MAKE_FUNCPTR(jack_port_name);
 MAKE_FUNCPTR(jack_get_buffer_size);
 #undef MAKE_FUNCPTR
 
+static void *jackhandle;
+
 /* define the below to work around a bug in jack where closing a port */
 /* takes a very long time, so to get around this we actually don't */
 /* close the port when the device is closed but instead mark the */
@@ -2396,28 +2395,53 @@ DWORD WINAPI JACK_widMessage(WORD wDevID, WORD wMsg, DWORD dwUser,
   return MMSYSERR_NOTSUPPORTED;
 }
 
-#else /* !SONAME_LIBJACK */
-
 /**************************************************************************
- * 				widMessage (WINEJACK.6)
+ * 				DriverProc (WINEJACK.1)
  */
-DWORD WINAPI JACK_widMessage(WORD wDevID, WORD wMsg, DWORD dwUser,
-                             DWORD_PTR dwParam1, DWORD_PTR dwParam2)
-{
-  FIXME("(%u, %04X, %08X, %08lX, %08lX):jack support not compiled into wine\n",
-        wDevID, wMsg, dwUser, dwParam1, dwParam2);
-  return MMSYSERR_NOTENABLED;
-}
-
-/**************************************************************************
- * 				wodMessage (WINEJACK.7)
- */
-DWORD WINAPI JACK_wodMessage(WORD wDevID, WORD wMsg, DWORD dwUser,
-                             DWORD_PTR dwParam1, DWORD_PTR dwParam2)
+LRESULT CALLBACK JACK_DriverProc(DWORD_PTR dwDevID, HDRVR hDriv, UINT wMsg, 
+                                 LPARAM dwParam1, LPARAM dwParam2)
 {
-  FIXME("(%u, %04X, %08X, %08lX, %08lX):jack support not compiled into wine\n",
-        wDevID, wMsg, dwUser, dwParam1, dwParam2);
-  return MMSYSERR_NOTENABLED;
+     TRACE("(%08lX, %p, %s (%08X), %08lX, %08lX)\n",
+           dwDevID, hDriv, wMsg == DRV_LOAD ? "DRV_LOAD" :
+           wMsg == DRV_FREE ? "DRV_FREE" :
+           wMsg == DRV_OPEN ? "DRV_OPEN" :
+           wMsg == DRV_CLOSE ? "DRV_CLOSE" :
+           wMsg == DRV_ENABLE ? "DRV_ENABLE" :
+           wMsg == DRV_DISABLE ? "DRV_DISABLE" :
+           wMsg == DRV_QUERYCONFIGURE ? "DRV_QUERYCONFIGURE" :
+           wMsg == DRV_CONFIGURE ? "DRV_CONFIGURE" :
+           wMsg == DRV_INSTALL ? "DRV_INSTALL" :
+           wMsg == DRV_REMOVE ? "DRV_REMOVE" : "UNKNOWN",
+           wMsg, dwParam1, dwParam2);
+
+    switch(wMsg) {
+    case DRV_LOAD:
+        jackhandle = wine_dlopen(SONAME_LIBJACK, RTLD_NOW, NULL, 0);
+        if (!jackhandle)
+        {
+            FIXME("error loading the jack library %s, please install this library to use jack\n",
+                  SONAME_LIBJACK);
+            return 0;
+        }
+        return 1;
+    case DRV_FREE:
+        if (jackhandle)
+        {
+            TRACE("calling wine_dlclose() on jackhandle\n");
+            wine_dlclose(jackhandle, NULL, 0);
+            jackhandle = NULL;
+        }
+        return 1;
+    case DRV_OPEN:
+    case DRV_CLOSE:
+    case DRV_INSTALL:
+    case DRV_REMOVE:
+    case DRV_ENABLE:
+    case DRV_DISABLE:
+    case DRV_QUERYCONFIGURE:
+        return 1;
+    case DRV_CONFIGURE:		MessageBoxA(0, "jack audio driver!", "jack driver", MB_OK);	return 1;
+    default:
+	return 0;
+    }
 }
-
-#endif /* SONAME_LIBJACK */
diff --git a/dlls/winejack.drv/jack.c b/dlls/winejack.drv/jack.c
deleted file mode 100644
index ebcb6c7..0000000
--- a/dlls/winejack.drv/jack.c
+++ /dev/null
@@ -1,122 +0,0 @@
-/* -*- tab-width: 8; c-basic-offset: 4 -*- */
-/*
- * Wine Driver for jack Sound Server
- *   http://jackit.sourceforge.net
- *
- * Copyright 2002 Chris Morgan
- *
- * This library is free software; you can redistribute it and/or
- * modify it under the terms of the GNU Lesser General Public
- * License as published by the Free Software Foundation; either
- * version 2.1 of the License, or (at your option) any later version.
- *
- * This library is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
- * Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public
- * License along with this library; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA
- */
-
-#include "config.h"
-#include "wine/port.h"
-
-#include <stdarg.h>
-#include <stdio.h>
-
-#include "windef.h"
-#include "winbase.h"
-#include "wingdi.h"
-#include "winuser.h"
-#include "mmddk.h"
-#include "jack.h"
-#include "wine/library.h"
-#include "wine/debug.h"
-
-WINE_DEFAULT_DEBUG_CHANNEL(jack);
-
-#ifdef SONAME_LIBJACK
-
-void *jackhandle = NULL;
-
-/**************************************************************************
- * 				JACK_drvLoad			[internal]	
- */
-static LRESULT JACK_drvLoad(void)
-{
-  TRACE("()\n");
-
-  /* dynamically load the jack library if not already loaded */
-  jackhandle = wine_dlopen(SONAME_LIBJACK, RTLD_NOW, NULL, 0);
-  TRACE("SONAME_LIBJACK == %s\n", SONAME_LIBJACK);
-  TRACE("jackhandle == %p\n", jackhandle);
-  if(!jackhandle)
-  {
-    FIXME("error loading the jack library %s, please install this library to use jack\n",
-          SONAME_LIBJACK);
-    return 0;
-  }
-
-  return 1;
-}
-
-/**************************************************************************
- * 				JACK_drvFree			[internal]	
- */
-/* unload the jack library on driver free */
-static LRESULT JACK_drvFree(void)
-{
-  TRACE("()\n");
-
-  if(jackhandle)
-  {
-    TRACE("calling wine_dlclose() on jackhandle\n");
-    wine_dlclose(jackhandle, NULL, 0);
-    jackhandle = NULL;
-  }
-
-  return 1;
-}
-
-#endif /* #ifdef SONAME_LIBJACK */
-
-
-/**************************************************************************
- * 				DriverProc (WINEJACK.1)
- */
-LRESULT CALLBACK JACK_DriverProc(DWORD_PTR dwDevID, HDRVR hDriv, UINT wMsg, 
-                                 LPARAM dwParam1, LPARAM dwParam2)
-{
-     TRACE("(%08lX, %p, %s (%08X), %08lX, %08lX)\n",
-           dwDevID, hDriv, wMsg == DRV_LOAD ? "DRV_LOAD" :
-           wMsg == DRV_FREE ? "DRV_FREE" :
-           wMsg == DRV_OPEN ? "DRV_OPEN" :
-           wMsg == DRV_CLOSE ? "DRV_CLOSE" :
-           wMsg == DRV_ENABLE ? "DRV_ENABLE" :
-           wMsg == DRV_DISABLE ? "DRV_DISABLE" :
-           wMsg == DRV_QUERYCONFIGURE ? "DRV_QUERYCONFIGURE" :
-           wMsg == DRV_CONFIGURE ? "DRV_CONFIGURE" :
-           wMsg == DRV_INSTALL ? "DRV_INSTALL" :
-           wMsg == DRV_REMOVE ? "DRV_REMOVE" : "UNKNOWN", 
-           wMsg, dwParam1, dwParam2);
-    
-    switch(wMsg) {
-#ifdef SONAME_LIBJACK
-    case DRV_LOAD:		return JACK_drvLoad();
-    case DRV_FREE:		return JACK_drvFree();
-    case DRV_OPEN:
-    case DRV_CLOSE:
-    case DRV_INSTALL:
-    case DRV_REMOVE:
-    case DRV_ENABLE:
-    case DRV_DISABLE:
-    case DRV_QUERYCONFIGURE:
-        return 1;
-    case DRV_CONFIGURE:		MessageBoxA(0, "jack audio driver!", "jack driver", MB_OK);	return 1;
-#endif
-    default:
-	return 0;
-    }
-}
diff --git a/dlls/winejack.drv/jack.h b/dlls/winejack.drv/jack.h
deleted file mode 100644
index 5545f24..0000000
--- a/dlls/winejack.drv/jack.h
+++ /dev/null
@@ -1,25 +0,0 @@
-/* Definition for jack driver : wine multimedia system
- *
- * Copyright 2002 Chris Morgan
- *
- * This library is free software; you can redistribute it and/or
- * modify it under the terms of the GNU Lesser General Public
- * License as published by the Free Software Foundation; either
- * version 2.1 of the License, or (at your option) any later version.
- *
- * This library is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
- * Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public
- * License along with this library; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA
- */
-
-#ifndef __WINE_JACK_H
-#define __WINE_JACK_H
-
-extern void *jackhandle; /* pointer to the libjack library */
-
-#endif /* __WINE_JACK_H */




More information about the wine-cvs mailing list