Re: winmm: Simplify MCI_DumpCommandTable a bit. (RETRY)

[Alexandre Julliard]

Gerald Pfeifer <gerald(a)pfeifer.com> writes:
> Ping.  Any objection to this, given that this has not been used for
> quite a while?

The variable is here for documentation, you don't want to remove it. You
can comment it out if it's not used.

-- 
Alexandre Julliard
julliard(a)winehq.org

winmm: Simplify MCI_DumpCommandTable a bit. (RETRY)

[Gerald Pfeifer]

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);