From: <rel...@us...> - 2009-09-27 19:56:10
|
Revision: 380 http://modplug.svn.sourceforge.net/modplug/?rev=380&view=rev Author: relabsoluness Date: 2009-09-27 19:55:54 +0000 (Sun, 27 Sep 2009) Log Message: ----------- [Fix] MIDI mapping: Pattern record checkbox wasn't updated properly. [Fix] Sample map: was broken when dealing with sample indexes greater than 255. [Mod] Version: Changed version number. [Ref] Minor code cleanup. Modified Paths: -------------- trunk/OpenMPT/mptrack/MIDIMappingDialog.cpp trunk/OpenMPT/mptrack/MainFrm.cpp trunk/OpenMPT/mptrack/Moddoc.cpp trunk/OpenMPT/mptrack/View_tre.cpp trunk/OpenMPT/mptrack/dlg_misc.cpp trunk/OpenMPT/mptrack/dlg_misc.h trunk/OpenMPT/mptrack/version.h Modified: trunk/OpenMPT/mptrack/MIDIMappingDialog.cpp =================================================================== --- trunk/OpenMPT/mptrack/MIDIMappingDialog.cpp 2009-09-24 15:24:32 UTC (rev 379) +++ trunk/OpenMPT/mptrack/MIDIMappingDialog.cpp 2009-09-27 19:55:54 UTC (rev 380) @@ -134,8 +134,9 @@ if(i < 0 || (size_t)i >= m_rMIDIMapper.GetCount()) return; m_Setting = m_rMIDIMapper.GetDirective(i); CMIDIMappingDirective& activeSetting = m_Setting; - CheckDlgButton(IDC_CHECKACTIVE, activeSetting.IsActive() ? MF_CHECKED : MF_UNCHECKED); - CheckDlgButton(IDC_CHECKCAPTURE, activeSetting.GetCaptureMIDI() ? MF_CHECKED : MF_UNCHECKED); + CheckDlgButton(IDC_CHECKACTIVE, activeSetting.IsActive() ? BST_CHECKED : BST_UNCHECKED); + CheckDlgButton(IDC_CHECKCAPTURE, activeSetting.GetCaptureMIDI() ? BST_CHECKED : BST_UNCHECKED); + CheckDlgButton(IDC_CHECK_PATRECORD, activeSetting.GetAllowPatternEdit() ? BST_CHECKED : BST_UNCHECKED); m_ChannelCBox.SetCurSel(activeSetting.GetChannel()); Modified: trunk/OpenMPT/mptrack/MainFrm.cpp =================================================================== --- trunk/OpenMPT/mptrack/MainFrm.cpp 2009-09-24 15:24:32 UTC (rev 379) +++ trunk/OpenMPT/mptrack/MainFrm.cpp 2009-09-27 19:55:54 UTC (rev 380) @@ -399,7 +399,7 @@ wsprintf(s, "Color%02d", ncol); rgbCustomColors[ncol] = GetPrivateProfileDWord("Display", s, rgbCustomColors[ncol], iniFile); } - if(rgbCustomColors[MODCOLOR_SEPSHADOW] == rgbCustomColors[MODCOLOR_SEPFACE] == rgbCustomColors[MODCOLOR_SEPHILITE] == 0 && vIniVersion < MAKE_VERSION_NUMERIC(1,17,03,02)) + if(rgbCustomColors[MODCOLOR_SEPSHADOW] == rgbCustomColors[MODCOLOR_SEPFACE] == rgbCustomColors[MODCOLOR_SEPHILITE] == 0 && vIniVersion < MAKE_VERSION_NUMERIC(1,17,03,03)) { rgbCustomColors[MODCOLOR_SEPSHADOW] = GetSysColor(COLOR_BTNSHADOW); rgbCustomColors[MODCOLOR_SEPFACE] = GetSysColor(COLOR_BTNFACE); Modified: trunk/OpenMPT/mptrack/Moddoc.cpp =================================================================== --- trunk/OpenMPT/mptrack/Moddoc.cpp 2009-09-24 15:24:32 UTC (rev 379) +++ trunk/OpenMPT/mptrack/Moddoc.cpp 2009-09-27 19:55:54 UTC (rev 380) @@ -3159,7 +3159,7 @@ END_CRITICAL(); // set playback timer in the status bar - SetElapsedTime(static_cast<ORDERINDEX>(nOrd), nRow, true); + SetElapsedTime(nOrd, nRow, true); if (pModPlaying != this) { @@ -3214,7 +3214,7 @@ END_CRITICAL(); // set playback timer in the status bar - SetElapsedTime(static_cast<ORDERINDEX>(nOrd), nRow, true); + SetElapsedTime(nOrd, nRow, true); if (pModPlaying != this) { pMainFrm->PlayMod(this, followSonghWnd, m_dwNotifyType|MPTNOTIFY_POSITION|MPTNOTIFY_VUMETERS); //rewbs.fix2977 @@ -3272,7 +3272,7 @@ END_CRITICAL(); // set playback timer in the status bar - SetElapsedTime(static_cast<ORDERINDEX>(nOrd), nRow, true); + SetElapsedTime(nOrd, nRow, true); if (pModPlaying != this) { pMainFrm->PlayMod(this, followSonghWnd, m_dwNotifyType|MPTNOTIFY_POSITION|MPTNOTIFY_VUMETERS); //rewbs.fix2977 Modified: trunk/OpenMPT/mptrack/View_tre.cpp =================================================================== --- trunk/OpenMPT/mptrack/View_tre.cpp 2009-09-24 15:24:32 UTC (rev 379) +++ trunk/OpenMPT/mptrack/View_tre.cpp 2009-09-27 19:55:54 UTC (rev 380) @@ -2080,10 +2080,10 @@ { UINT nDefault = 0; BOOL bSep = FALSE; - DWORD dwItemType, dwItemNo; + DWORD dwItemType; dwItemType = GetModItem(hItem); - dwItemNo = dwItemType >> 16; + const uint16 nItemNo = HIWORD(dwItemType); dwItemType &= 0xFFFF; SelectItem(hItem); switch(dwItemType) @@ -2111,7 +2111,7 @@ if ((pModDoc) && (!pModDoc->GetNumInstruments())) { AppendMenu(hMenu, MF_SEPARATOR, NULL, ""); - AppendMenu(hMenu, (pModDoc->IsSampleMuted(dwItemNo) ? MF_CHECKED:0)|MF_STRING, ID_MODTREE_MUTE, "&Mute Sample"); + AppendMenu(hMenu, (pModDoc->IsSampleMuted(nItemNo) ? MF_CHECKED:0)|MF_STRING, ID_MODTREE_MUTE, "&Mute Sample"); AppendMenu(hMenu, MF_STRING, ID_MODTREE_SOLO, "&Solo Sample"); AppendMenu(hMenu, MF_STRING, ID_MODTREE_UNMUTEALL, "&Unmute all"); } @@ -2128,7 +2128,7 @@ if ((pModDoc) && (pModDoc->GetNumInstruments())) { AppendMenu(hMenu, MF_SEPARATOR, NULL, ""); - AppendMenu(hMenu, (pModDoc->IsInstrumentMuted(dwItemNo) ? MF_CHECKED:0)|MF_STRING, ID_MODTREE_MUTE, "&Mute Instrument"); + AppendMenu(hMenu, (pModDoc->IsInstrumentMuted(nItemNo) ? MF_CHECKED:0)|MF_STRING, ID_MODTREE_MUTE, "&Mute Instrument"); AppendMenu(hMenu, MF_STRING, ID_MODTREE_SOLO, "&Solo Instrument"); AppendMenu(hMenu, MF_STRING, ID_MODTREE_UNMUTEALL, "&Unmute all"); // -> CODE#0023 @@ -2149,7 +2149,7 @@ CModDoc *pModDoc = GetDocumentFromItem(hItem); CSoundFile *pSndFile = pModDoc ? pModDoc->GetSoundFile() : NULL; if (pSndFile) { - PSNDMIXPLUGIN pPlugin = &pSndFile->m_MixPlugins[dwItemNo]; + PSNDMIXPLUGIN pPlugin = &pSndFile->m_MixPlugins[nItemNo]; if (pPlugin) { bool bypassed = ((pPlugin->Info.dwInputRouting&MIXPLUG_INPUTF_BYPASS) != 0); AppendMenu(hMenu, (bypassed?MF_CHECKED:0)|MF_STRING, ID_MODTREE_MUTE, "&Bypass"); @@ -2490,29 +2490,29 @@ //----------------------------- { HTREEITEM hItem = GetSelectedItem(); - DWORD dwItemType, dwItemNo; + DWORD dwItemType; CModDoc *pModDoc; dwItemType = GetModItem(hItem); - dwItemNo = dwItemType >> 16; + const uint16 nItemNo = HIWORD(dwItemType); dwItemType &= 0xFFFF; pModDoc = GetDocumentFromItem(hItem); if (pModDoc) { if ((dwItemType == MODITEM_SAMPLE) && (!pModDoc->GetNumInstruments())) { - pModDoc->MuteSample(dwItemNo, (pModDoc->IsSampleMuted(dwItemNo)) ? FALSE : TRUE); + pModDoc->MuteSample(nItemNo, (pModDoc->IsSampleMuted(nItemNo)) ? FALSE : TRUE); } else if ((dwItemType == MODITEM_INSTRUMENT) && (pModDoc->GetNumInstruments())) { - pModDoc->MuteInstrument(dwItemNo, (pModDoc->IsInstrumentMuted(dwItemNo)) ? FALSE : TRUE); + pModDoc->MuteInstrument(nItemNo, (pModDoc->IsInstrumentMuted(nItemNo)) ? FALSE : TRUE); } if ((dwItemType == MODITEM_EFFECT)) { CSoundFile *pSndFile = pModDoc ? pModDoc->GetSoundFile() : NULL; if (pSndFile) { - PSNDMIXPLUGIN pPlugin = &pSndFile->m_MixPlugins[dwItemNo]; + PSNDMIXPLUGIN pPlugin = &pSndFile->m_MixPlugins[nItemNo]; if (pPlugin) { CVstPlugin *pVstPlugin = (CVstPlugin *)pPlugin->pMixPlugin; if (pVstPlugin) pVstPlugin->Bypass(); @@ -2528,11 +2528,11 @@ //----------------------------- { HTREEITEM hItem = GetSelectedItem(); - DWORD dwItemType, dwItemNo; + DWORD dwItemType; CModDoc *pModDoc; dwItemType = GetModItem(hItem); - dwItemNo = dwItemType >> 16; + const uint16 nItemNo = HIWORD(dwItemType); dwItemType &= 0xFFFF; pModDoc = GetDocumentFromItem(hItem); if (pModDoc) @@ -2541,16 +2541,16 @@ INSTRUMENTINDEX nInstruments = pModDoc->GetNumInstruments(); if ((dwItemType == MODITEM_SAMPLE) && (!nInstruments)) { - for (UINT i=1; i<=nSamples; i++) + for (SAMPLEINDEX i=1; i<=nSamples; i++) { - pModDoc->MuteSample(i, (i == dwItemNo) ? FALSE : TRUE); + pModDoc->MuteSample(i, (i == nItemNo) ? FALSE : TRUE); } } else if ((dwItemType == MODITEM_INSTRUMENT) && (nInstruments)) { - for (UINT i=1; i<=nInstruments; i++) + for (INSTRUMENTINDEX i=1; i<=nInstruments; i++) { - pModDoc->MuteInstrument(i, (i == dwItemNo) ? FALSE : TRUE); + pModDoc->MuteInstrument(i, (i == nItemNo) ? FALSE : TRUE); } } } @@ -2572,11 +2572,11 @@ INSTRUMENTINDEX nInstruments = pModDoc->GetNumInstruments(); if ((dwItemType == MODITEM_SAMPLE) || (dwItemType == MODITEM_INSTRUMENT)) { - for (UINT i=1; i<=nSamples; i++) + for (SAMPLEINDEX i=1; i<=nSamples; i++) { pModDoc->MuteSample(i, FALSE); } - for (UINT j=1; j<=nInstruments; j++) + for (INSTRUMENTINDEX j=1; j<=nInstruments; j++) { pModDoc->MuteInstrument(j, FALSE); } @@ -2623,14 +2623,14 @@ { HTREEITEM hItem = GetSelectedItem(); DWORD dwItemType = GetModItem(hItem); - DWORD dwItem = dwItemType >> 16; + const uint16 nItem = HIWORD(dwItemType); dwItemType &= 0xFFFF; CModDoc *pModDoc = GetDocumentFromItem(hItem); CSoundFile *pSndFile = pModDoc ? pModDoc->GetSoundFile() : NULL; - if(pSndFile && dwItem){ + if(pSndFile && nItem){ - if(pSndFile->m_szInstrumentPath[dwItem-1][0] == '\0'){ + if(pSndFile->m_szInstrumentPath[nItem-1][0] == '\0'){ CHAR pszFileNames[_MAX_PATH]; CFileDialog dlg(FALSE, (pSndFile->m_nType & (MOD_TYPE_IT|MOD_TYPE_MPT)) ? "iti" : "xi", NULL, @@ -2646,23 +2646,23 @@ dlg.m_ofn.lpstrFile = pszFileNames; dlg.m_ofn.nMaxFile = _MAX_PATH; - if(dlg.DoModal() == IDOK) strcpy(pSndFile->m_szInstrumentPath[dwItem-1], pszFileNames); + if(dlg.DoModal() == IDOK) strcpy(pSndFile->m_szInstrumentPath[nItem-1], pszFileNames); dlg.m_ofn.lpstrFile = NULL; dlg.m_ofn.nMaxFile = 0; } - if(pSndFile->m_szInstrumentPath[dwItem-1][0] != '\0'){ - int size = strlen(pSndFile->m_szInstrumentPath[dwItem-1]); - BOOL iti = _stricmp(&pSndFile->m_szInstrumentPath[dwItem-1][size-3],"iti") == 0; - BOOL xi = _stricmp(&pSndFile->m_szInstrumentPath[dwItem-1][size-2],"xi") == 0; + if(pSndFile->m_szInstrumentPath[nItem-1][0] != '\0'){ + int size = strlen(pSndFile->m_szInstrumentPath[nItem-1]); + BOOL iti = _stricmp(&pSndFile->m_szInstrumentPath[nItem-1][size-3],"iti") == 0; + BOOL xi = _stricmp(&pSndFile->m_szInstrumentPath[nItem-1][size-2],"xi") == 0; if(iti || (!iti && !xi && pSndFile->m_nType & (MOD_TYPE_IT|MOD_TYPE_MPT))) - pSndFile->SaveITIInstrument(dwItem, pSndFile->m_szInstrumentPath[dwItem-1]); + pSndFile->SaveITIInstrument(nItem, pSndFile->m_szInstrumentPath[nItem-1]); if(xi || (!xi && !iti && pSndFile->m_nType == MOD_TYPE_XM)) - pSndFile->SaveXIInstrument(dwItem, pSndFile->m_szInstrumentPath[dwItem-1]); + pSndFile->SaveXIInstrument(nItem, pSndFile->m_szInstrumentPath[nItem-1]); - pSndFile->instrumentModified[dwItem-1] = FALSE; + pSndFile->instrumentModified[nItem-1] = FALSE; } if(pModDoc) pModDoc->UpdateAllViews(NULL, HINT_MODTYPE); Modified: trunk/OpenMPT/mptrack/dlg_misc.cpp =================================================================== --- trunk/OpenMPT/mptrack/dlg_misc.cpp 2009-09-24 15:24:32 UTC (rev 379) +++ trunk/OpenMPT/mptrack/dlg_misc.cpp 2009-09-27 19:55:54 UTC (rev 380) @@ -2787,7 +2787,7 @@ KeyboardMap[iNote] = pIns->Keyboard[iNote]; } else { - KeyboardMap[iNote] = (BYTE)nSample; + KeyboardMap[iNote] = (WORD)nSample; } /* rewbs.note: I don't think we need this with cust keys. // -> CODE#0009 Modified: trunk/OpenMPT/mptrack/dlg_misc.h =================================================================== --- trunk/OpenMPT/mptrack/dlg_misc.h 2009-09-24 15:24:32 UTC (rev 379) +++ trunk/OpenMPT/mptrack/dlg_misc.h 2009-09-27 19:55:54 UTC (rev 380) @@ -535,7 +535,7 @@ CSliderCtrl m_SbOctave; CSoundFile *m_pSndFile; UINT m_nInstrument; - BYTE KeyboardMap[NOTE_MAX]; + WORD KeyboardMap[NOTE_MAX]; public: CSampleMapDlg(CSoundFile *pSndFile, UINT nInstr, CWnd *parent=NULL):CDialog(IDD_EDITSAMPLEMAP, parent) Modified: trunk/OpenMPT/mptrack/version.h =================================================================== --- trunk/OpenMPT/mptrack/version.h 2009-09-24 15:24:32 UTC (rev 379) +++ trunk/OpenMPT/mptrack/version.h 2009-09-27 19:55:54 UTC (rev 380) @@ -15,7 +15,7 @@ #define VER_MAJORMAJOR 1 #define VER_MAJOR 17 #define VER_MINOR 03 -#define VER_MINORMINOR 02 +#define VER_MINORMINOR 03 //Creates version number from version parts that appears in version string. //For example MAKE_VERSION_NUMERIC(1,17,02,28) gives version number of This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. |