winmm: Simplify MCI_DumpCommandTable a bit. (RETRY)

Gerald Pfeifer gerald at pfeifer.com
Thu Feb 17 21:03:01 CST 2011


On Wed, 16 Feb 2011, Alexandre Julliard wrote:
> The variable is here for documentation, you don't want to remove it. You 
> can comment it out if it's not used.

Fair enough!  Updated patch below.

Gerald


ChangeLog:
winmm: Comment out unused variable in Simplify MCI_DumpCommandTable.

diff --git a/dlls/winmm/mci.c b/dlls/winmm/mci.c
index 30db363..8a889d2 100644
--- a/dlls/winmm/mci.c
+++ b/dlls/winmm/mci.c
@@ -625,7 +625,6 @@ static	BOOL		MCI_DumpCommandTable(UINT uTbl)
 {
     const BYTE*	lmem;
     LPCWSTR	str;
-    DWORD	flg;
     WORD	eid;
 
     if (!MCI_IsCommandTableValid(uTbl)) {
@@ -636,9 +635,10 @@ static	BOOL		MCI_DumpCommandTable(UINT uTbl)
     lmem = S_MciCmdTable[uTbl].lpTable;
     do {
 	do {
+	    /* DWORD flg; */
 	    str = (LPCWSTR)lmem;
 	    lmem += (strlenW(str) + 1) * sizeof(WCHAR);
-	    flg = *(const DWORD*)lmem;
+	    /* flg = *(const DWORD*)lmem; */
 	    eid = *(const WORD*)(lmem + sizeof(DWORD));
             /* TRACE("cmd=%s %08lx %04x\n", debugstr_w(str), flg, eid); */
 	    lmem += sizeof(DWORD) + sizeof(WORD);



More information about the wine-devel mailing list