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