commit d34abdd15830e6e6da43fddc46aa75fb52574c2c
author Eric Pouech <Eric.Pouech@wanadoo.fr> Sat Apr 29 14:19:02 2000 +0000
committer Alexandre Julliard <julliard@winehq.org> Sat Apr 29 14:19:02 2000 +0000
tree 8f1b095812164c62c037ffff265cd98407975c83
parent ec4f57bec1834cc75a50626b2a21d218813ddf11

Fixed bug when file doesn't exist in MCI_OPEN handling.
Reduced time spent with crit sect locked.

dlls/winmm/mci.c

diff --git a/dlls/winmm/mci.c b/dlls/winmm/mci.c
index 39b4624..a2cd866 100644
--- a/dlls/winmm/mci.c
+++ b/dlls/winmm/mci.c
@@ -427,6 +427,10 @@
 	dwRet = MCIERR_OUT_OF_MEMORY;
 	goto errCleanUp;
     }
+ 
+    wmd->lpfnYieldProc = MCI_DefYieldProc;
+    wmd->dwYieldData = VK_CANCEL;
+    wmd->hCreatorTask = GetCurrentTask();
 
     EnterCriticalSection(&iData->cs);
     /* wmd must be inserted in list before sending opening the driver, coz' it
@@ -440,9 +444,6 @@
 	 modp.wDeviceID++);
 
     wmd->wDeviceID = modp.wDeviceID;
-    wmd->lpfnYieldProc = MCI_DefYieldProc;
-    wmd->dwYieldData = VK_CANCEL;
-    wmd->hCreatorTask = GetCurrentTask();
 
     LeaveCriticalSection(&iData->cs);
 
@@ -800,6 +801,7 @@
     DWORD		data[MCI_DATA_SIZE];
     LPCSTR		lpCmd = 0;
     LPWINE_MM_IDATA	iData = MULTIMEDIA_GetIData();
+    BOOL		bAutoOpen = FALSE;
 
     TRACE("('%s', %p, %d, %X)\n", lpstrCommand, lpstrRet, uRetLen, hwndCallback);
 
@@ -860,11 +862,17 @@
 	    MCI_UnLoadMciDriver(iData, wmd);
 	    goto errCleanUp;
 	}
-    } else {
+    } else if (!(wmd = MCI_GetDriver(mciGetDeviceIDA(dev)))) {
+	/* auto open */
+	char	buf[128];
+	sprintf(buf, "open %s wait", dev);
+	
+	if ((dwRet = mciSendStringA(buf, NULL, 0, 0)) != 0)
+	    goto errCleanUp;
+	
 	wmd = MCI_GetDriver(mciGetDeviceIDA(dev));
 	if (!wmd) {
-	    FIXME("Oooch: couldn't find driver for '%s' in '%s'; automatic open is NIY\n", 
-		  dev, lpstrCommand);
+	    /* FIXME: memory leak, MCI driver is not closed */
 	    dwRet = MCIERR_INVALID_DEVICE_ID;
 	    goto errCleanUp;
 	}
@@ -909,6 +917,13 @@
     if ((dwRet = MCI_ParseOptArgs(data, offset, lpCmd, args, &dwFlags)))
 	goto errCleanUp;
 
+    if (bAutoOpen && (dwFlags & MCI_NOTIFY)) {
+	dwRet = MCIERR_NOTIFY_ON_AUTO_OPEN;
+	goto errCleanUp;
+    }
+    /* FIXME: the command should get it's own notification window set up and
+     * ask for device closing while processing the notification mechanism
+     */
     if (lpstrRet && uRetLen) *lpstrRet = '\0';
 
 #define	STR_OF(_x) (IsBadReadPtr((char*)_x,1)?"?":(char*)(_x))
@@ -2012,7 +2027,7 @@
 {
     char			strDevTyp[128];
     DWORD 			dwRet; 
-    LPWINE_MCIDRIVER		wmd;
+    LPWINE_MCIDRIVER		wmd = NULL;
     LPWINE_MM_IDATA		iData = MULTIMEDIA_GetIData();
 
     TRACE("(%08lX, %p)\n", dwParam, lpParms);
@@ -2127,7 +2142,7 @@
     
     return 0;
 errCleanUp:
-    MCI_UnLoadMciDriver(iData, wmd);
+    if (wmd) MCI_UnLoadMciDriver(iData, wmd);
 
     if (dwParam & MCI_NOTIFY)
 	mciDriverNotify16(lpParms->dwCallback, 0, MCI_NOTIFY_FAILURE);