--- linuxsampler/trunk/src/engines/EngineBase.h 2010/08/12 15:36:15 2115 +++ linuxsampler/trunk/src/engines/EngineBase.h 2016/07/16 11:24:39 2953 @@ -4,7 +4,8 @@ * * * Copyright (C) 2003,2004 by Benno Senoner and Christian Schoenebeck * * Copyright (C) 2005-2008 Christian Schoenebeck * - * Copyright (C) 2009-2010 Christian Schoenebeck and Grigor Iliev * + * Copyright (C) 2009-2012 Christian Schoenebeck and Grigor Iliev * + * Copyright (C) 2012-2016 Christian Schoenebeck and Andreas Persson * * * * This program is free software; you can redistribute it and/or modify * * it under the terms of the GNU General Public License as published by * @@ -32,6 +33,9 @@ #include "InstrumentManager.h" #include "../common/global_private.h" +// a bit headroom over CONFIG_MAX_VOICES to avoid minor complications i.e. under voice stealing conditions +#define MAX_NOTES_HEADROOM 3 +#define GLOBAL_MAX_NOTES (GLOBAL_MAX_VOICES * MAX_NOTES_HEADROOM) namespace LinuxSampler { @@ -45,27 +49,41 @@ class IM /* Instrument Manager */, class I /* Instrument */ > - class EngineBase: public AbstractEngine, public RegionPools, public VoicePool { + class EngineBase: public AbstractEngine, public RegionPools, public NotePool { public: + typedef typename RTList< Note >::Iterator NoteIterator; typedef typename RTList::Iterator VoiceIterator; typedef typename Pool::Iterator PoolVoiceIterator; typedef typename RTList::Iterator RootRegionIterator; typedef typename MidiKeyboardManager::MidiKey MidiKey; - EngineBase() : SuspendedRegions(128) { + EngineBase() : SuspendedRegions(128), noteIDPool(GLOBAL_MAX_NOTES) { pDiskThread = NULL; + pNotePool = new Pool< Note >(GLOBAL_MAX_NOTES); + pNotePool->setPoolElementIDsReservedBits(INSTR_SCRIPT_EVENT_ID_RESERVED_BITS); pVoicePool = new Pool(GLOBAL_MAX_VOICES); pRegionPool[0] = new Pool(GLOBAL_MAX_VOICES); pRegionPool[1] = new Pool(GLOBAL_MAX_VOICES); pVoiceStealingQueue = new RTList(pEventPool); iMaxDiskStreams = GLOBAL_MAX_STREAMS; - for (VoiceIterator iterVoice = pVoicePool->allocAppend(); iterVoice == pVoicePool->last(); iterVoice = pVoicePool->allocAppend()) { + // init all Voice objects in voice pool + for (VoiceIterator iterVoice = pVoicePool->allocAppend(); + iterVoice; iterVoice = pVoicePool->allocAppend()) + { iterVoice->SetEngine(this); } pVoicePool->clear(); + // init all Note objects in note pool + for (NoteIterator itNote = pNotePool->allocAppend(); itNote; + itNote = pNotePool->allocAppend()) + { + itNote->init(pVoicePool, ¬eIDPool); + } + pNotePool->clear(); + ResetInternal(); ResetScaleTuning(); ResetSuspendedRegions(); @@ -79,6 +97,11 @@ dmsg(1,("OK\n")); } + if (pNotePool) { + pNotePool->clear(); + delete pNotePool; + } + if (pVoicePool) { pVoicePool->clear(); delete pVoicePool; @@ -104,7 +127,7 @@ * @param Samples - number of sample points to be rendered * @returns 0 on success */ - virtual int RenderAudio(uint Samples) { + virtual int RenderAudio(uint Samples) OVERRIDE { dmsg(8,("RenderAudio(Samples=%d)\n", Samples)); // return if engine disabled @@ -143,6 +166,10 @@ } } } + + // In case scale tuning has been changed, recalculate pitch for + // all active voices. + ProcessScaleTuningChange(); // reset internal voice counter (just for statistic of active voices) ActiveVoiceCountTemp = 0; @@ -190,15 +217,24 @@ // been deleted by the disk thread if (iPendingStreamDeletions) ProcessPendingStreamDeletions(); + // Release the instrument change command. (This has to + // be done after all voices have been rendered and not + // in HandleInstrumentChanges, as the RegionsInUse + // list has been built up by the voice renderers.) + for (int i = 0; i < engineChannels.size(); i++) { + EngineChannelBase* channel = + static_cast*>(engineChannels[i]); + channel->InstrumentChangeCommandReader.Unlock(); + } FrameTime += Samples; EngineDisabled.RttDone(); return 0; } - virtual int MaxVoices() { return pVoicePool->poolSize(); } + virtual int MaxVoices() OVERRIDE { return pVoicePool->poolSize(); } - virtual void SetMaxVoices(int iVoices) throw (Exception) { + virtual void SetMaxVoices(int iVoices) throw (Exception) OVERRIDE { if (iVoices < 1) throw Exception("Maximum voices for an engine cannot be set lower than 1"); @@ -222,26 +258,42 @@ pChannel->ResetRegionsInUse(pRegionPool); } + // FIXME: Shouldn't all those pool elements be freed before resizing the pools? try { pVoicePool->resizePool(iVoices); + pNotePool->resizePool(iVoices * MAX_NOTES_HEADROOM); + noteIDPool.resizePool(iVoices * MAX_NOTES_HEADROOM); } catch (...) { throw Exception("FATAL: Could not resize voice pool!"); } - for (VoiceIterator iterVoice = pVoicePool->allocAppend(); iterVoice == pVoicePool->last(); iterVoice = pVoicePool->allocAppend()) { + for (VoiceIterator iterVoice = pVoicePool->allocAppend(); + iterVoice; iterVoice = pVoicePool->allocAppend()) + { iterVoice->SetEngine(this); iterVoice->pDiskThread = this->pDiskThread; } pVoicePool->clear(); + for (NoteIterator itNote = pNotePool->allocAppend(); itNote; + itNote = pNotePool->allocAppend()) + { + itNote->init(pVoicePool, ¬eIDPool); + } + pNotePool->clear(); + + PostSetMaxVoices(iVoices); ResumeAll(); } + + /** Called after the new max number of voices is set and before resuming the engine. */ + virtual void PostSetMaxVoices(int iVoices) { } - virtual uint DiskStreamCount() { return (pDiskThread) ? pDiskThread->GetActiveStreamCount() : 0; } - virtual uint DiskStreamCountMax() { return (pDiskThread) ? pDiskThread->ActiveStreamCountMax : 0; } - virtual int MaxDiskStreams() { return iMaxDiskStreams; } + virtual uint DiskStreamCount() OVERRIDE { return (pDiskThread) ? pDiskThread->GetActiveStreamCount() : 0; } + virtual uint DiskStreamCountMax() OVERRIDE { return (pDiskThread) ? pDiskThread->ActiveStreamCountMax : 0; } + virtual int MaxDiskStreams() OVERRIDE { return iMaxDiskStreams; } - virtual void SetMaxDiskStreams(int iStreams) throw (Exception) { + virtual void SetMaxDiskStreams(int iStreams) throw (Exception) OVERRIDE { if (iStreams < 0) throw Exception("Maximum disk streams for an engine cannot be set lower than 0"); @@ -256,9 +308,9 @@ ResumeAll(); } - virtual String DiskStreamBufferFillBytes() { return (pDiskThread) ? pDiskThread->GetBufferFillBytes() : ""; } - virtual String DiskStreamBufferFillPercentage() { return (pDiskThread) ? pDiskThread->GetBufferFillPercentage() : ""; } - virtual InstrumentManager* GetInstrumentManager() { return &instruments; } + virtual String DiskStreamBufferFillBytes() OVERRIDE { return (pDiskThread) ? pDiskThread->GetBufferFillBytes() : ""; } + virtual String DiskStreamBufferFillPercentage() OVERRIDE { return (pDiskThread) ? pDiskThread->GetBufferFillPercentage() : ""; } + virtual InstrumentManager* GetInstrumentManager() OVERRIDE { return &instruments; } /** * Connect this engine instance with the given audio output device. @@ -269,7 +321,7 @@ * * @param pAudioOut - audio output device to connect to */ - virtual void Connect(AudioOutputDevice* pAudioOut) { + virtual void Connect(AudioOutputDevice* pAudioOut) OVERRIDE { // caution: don't ignore if connecting to the same device here, // because otherwise SetMaxDiskStreams() implementation won't work anymore! @@ -298,8 +350,9 @@ MinFadeOutSamples = MaxSamplesPerCycle; // lower minimum release time const float minReleaseTime = (float) MaxSamplesPerCycle / (float) SampleRate; + pVoicePool->clear(); for (VoiceIterator iterVoice = pVoicePool->allocAppend(); iterVoice == pVoicePool->last(); iterVoice = pVoicePool->allocAppend()) { - iterVoice->pEG1->CalculateFadeOutCoeff(minReleaseTime, SampleRate); + iterVoice->CalculateFadeOutCoeff(minReleaseTime, SampleRate); } pVoicePool->clear(); } @@ -318,6 +371,7 @@ exit(EXIT_FAILURE); } + pVoicePool->clear(); for (VoiceIterator iterVoice = pVoicePool->allocAppend(); iterVoice == pVoicePool->last(); iterVoice = pVoicePool->allocAppend()) { iterVoice->pDiskThread = this->pDiskThread; dmsg(3,("d")); @@ -332,13 +386,35 @@ pDiskThread->StartThread(); dmsg(1,("OK\n")); + bool printEqInfo = true; for (VoiceIterator iterVoice = pVoicePool->allocAppend(); iterVoice == pVoicePool->last(); iterVoice = pVoicePool->allocAppend()) { if (!iterVoice->pDiskThread) { dmsg(0,("Engine -> voice::trigger: !pDiskThread\n")); exit(EXIT_FAILURE); } + + iterVoice->CreateEq(); + + if(printEqInfo) { + iterVoice->PrintEqInfo(); + printEqInfo = false; + } } pVoicePool->clear(); + + // (re)create dedicated voice audio buffers + //TODO: we could optimize resource usage a bit by just allocating these dedicated voice buffers when there is at least one engine channel with FX sends, because only in this case those special buffers are used actually, but since it would usually only save couple bytes in total, its probably not worth it + if (pDedicatedVoiceChannelLeft) delete pDedicatedVoiceChannelLeft; + if (pDedicatedVoiceChannelRight) delete pDedicatedVoiceChannelRight; + pDedicatedVoiceChannelLeft = new AudioChannel(0, MaxSamplesPerCycle); + pDedicatedVoiceChannelRight = new AudioChannel(1, MaxSamplesPerCycle); + } + + // Implementattion for abstract method derived from Engine. + virtual void ReconnectAudioOutputDevice() OVERRIDE { + SuspendAll(); + if (pAudioOutputDevice) Connect(pAudioOutputDevice); + ResumeAll(); } /** @@ -393,13 +469,14 @@ * @param pRegion - region the engine shall stop using */ virtual void Suspend(RR* pRegion) { - dmsg(2,("EngineBase: Suspending Region %x ...\n",pRegion)); - SuspendedRegionsMutex.Lock(); - SuspensionChangeOngoing.Set(true); - pPendingRegionSuspension = pRegion; - SuspensionChangeOngoing.WaitAndUnlockIf(true); - SuspendedRegionsMutex.Unlock(); - dmsg(2,("EngineBase: Region %x suspended.",pRegion)); + dmsg(2,("EngineBase: Suspending Region %p ...\n",(void*)pRegion)); + { + LockGuard lock(SuspendedRegionsMutex); + SuspensionChangeOngoing.Set(true); + pPendingRegionSuspension = pRegion; + SuspensionChangeOngoing.WaitAndUnlockIf(true); + } + dmsg(2,("EngineBase: Region %p suspended.",(void*)pRegion)); } /** @@ -409,13 +486,14 @@ * @param pRegion - region the engine shall be allowed to use again */ virtual void Resume(RR* pRegion) { - dmsg(2,("EngineBase: Resuming Region %x ...\n",pRegion)); - SuspendedRegionsMutex.Lock(); - SuspensionChangeOngoing.Set(true); - pPendingRegionResumption = pRegion; - SuspensionChangeOngoing.WaitAndUnlockIf(true); - SuspendedRegionsMutex.Unlock(); - dmsg(2,("EngineBase: Region %x resumed.\n",pRegion)); + dmsg(2,("EngineBase: Resuming Region %p ...\n",(void*)pRegion)); + { + LockGuard lock(SuspendedRegionsMutex); + SuspensionChangeOngoing.Set(true); + pPendingRegionResumption = pRegion; + SuspensionChangeOngoing.WaitAndUnlockIf(true); + } + dmsg(2,("EngineBase: Region %p resumed.\n",(void*)pRegion)); } virtual void ResetSuspendedRegions() { @@ -523,32 +601,38 @@ return pRegionPool[index]; } - // implementation of abstract method derived from class 'LinuxSampler::VoicePool' - virtual Pool* GetVoicePool() { return pVoicePool; } + // implementation of abstract methods derived from class 'LinuxSampler::NotePool' + virtual Pool* GetVoicePool() OVERRIDE { return pVoicePool; } + virtual Pool< Note >* GetNotePool() OVERRIDE { return pNotePool; } + virtual Pool* GetNodeIDPool() OVERRIDE { return ¬eIDPool; } D* GetDiskThread() { return pDiskThread; } //friend class EngineChannelBase; + static IM instruments; + protected: class SuspensionVoiceHandler : public MidiKeyboardManager::VoiceHandler { public: int PendingStreamDeletions; RR* pPendingRegionSuspension; + SuspensionVoiceHandler(RR* pPendingRegionSuspension) { PendingStreamDeletions = 0; this->pPendingRegionSuspension = pPendingRegionSuspension; } - virtual bool Process(MidiKey* pMidiKey) { - VoiceIterator itVoice = pMidiKey->pActiveVoices->first(); + virtual bool Process(MidiKey* pMidiKey) OVERRIDE { + NoteIterator itNote = pMidiKey->pActiveNotes->first(); + VoiceIterator itVoice = itNote->pActiveVoices->first(); // if current key is not associated with this region, skip this key if (itVoice->GetRegion()->GetParent() != pPendingRegionSuspension) return false; return true; } - virtual void Process(VoiceIterator& itVoice) { + virtual void Process(VoiceIterator& itVoice) OVERRIDE { // request a notification from disk thread side for stream deletion const Stream::Handle hStream = itVoice->KillImmediately(true); if (hStream != Stream::INVALID_HANDLE) { // voice actually used a stream @@ -558,22 +642,104 @@ } }; - static IM instruments; - Pool* pRegionPool[2]; ///< Double buffered pool, used by the engine channels to keep track of regions in use. int MinFadeOutSamples; ///< The number of samples needed to make an instant fade out (e.g. for voice stealing) without leading to clicks. D* pDiskThread; int ActiveVoiceCountTemp; ///< number of currently active voices (for internal usage, will be used for incrementation) VoiceIterator itLastStolenVoice; ///< Only for voice stealing: points to the last voice which was theft in current audio fragment, NULL otherwise. + NoteIterator itLastStolenNote; ///< Only for voice stealing: points to the last note from which was theft in current audio fragment, NULL otherwise. RTList::Iterator iuiLastStolenKey; ///< Only for voice stealing: key number of last key on which the last voice was theft in current audio fragment, NULL otherwise. EngineChannelBase* pLastStolenChannel; ///< Only for voice stealing: points to the engine channel on which the previous voice was stolen in this audio fragment. VoiceIterator itLastStolenVoiceGlobally; ///< Same as itLastStolenVoice, but engine globally + NoteIterator itLastStolenNoteGlobally; ///< Same as itLastStolenNote, but engine globally RTList::Iterator iuiLastStolenKeyGlobally; ///< Same as iuiLastStolenKey, but engine globally RTList* pVoiceStealingQueue; ///< All voice-launching events which had to be postponed due to free voice shortage. Mutex ResetInternalMutex; ///< Mutex to protect the ResetInternal function for concurrent usage (e.g. by the lscp and instrument loader threads). int iMaxDiskStreams; + NoteBase* NoteByID(note_id_t id) OVERRIDE { + NoteIterator itNote = GetNotePool()->fromID(id); + if (!itNote) return NULL; + return &*itNote; + } + + /** + * Gets a new @c Note object from the note pool, initializes it + * appropriately, links it with requested parent note (if + * requested), moves it to the appropriate key's list of active + * notes it, and sticks the new note's unique ID to the + * passed @a pNoteOnEvent. + * + * @param pEngineChannel - engine channel on which this event happened + * @param pNoteOnEvent - event which caused this + * @returns new note's unique ID (or zero on error) + */ + note_id_t LaunchNewNote(LinuxSampler::EngineChannel* pEngineChannel, Event* pNoteOnEvent) OVERRIDE { + EngineChannelBase* pChannel = static_cast*>(pEngineChannel); + Pool< Note >* pNotePool = GetNotePool(); + + if (pNotePool->poolIsEmpty()) { + dmsg(1,("Engine: Could not launch new note; Note pool empty!\n")); + return 0; // error + } + + // create a new note (for new voices to be assigned to) + //NoteIterator itNewNote = pKey->pActiveNotes->allocAppend(); + NoteIterator itNewNote = pNotePool->allocAppend(); + const note_id_t newNoteID = pNotePool->getID(itNewNote); + + // usually the new note (and its subsequent voices) will be + // allocated on the key provided by the event's note number, + // however if this new note is requested not to be a regular + // note, but rather a child note, then this new note will be + // allocated on the parent note's key instead in order to + // release the child note simultaniously with its parent note + itNewNote->hostKey = pNoteOnEvent->Param.Note.Key; + + // in case this new note was requested to be a child note, + // then retrieve its parent note and link them with each other + const note_id_t parentNoteID = pNoteOnEvent->Param.Note.ParentNoteID; + if (parentNoteID) { + NoteIterator itParentNote = pNotePool->fromID(parentNoteID); + if (itParentNote) { + RTList::Iterator itChildNoteID = itParentNote->pChildNotes->allocAppend(); + if (itChildNoteID) { + // link parent and child note with each other + *itChildNoteID = newNoteID; + itNewNote->parentNoteID = parentNoteID; + itNewNote->hostKey = itParentNote->hostKey; + } else { + dmsg(1,("Engine: Could not assign new note as child note; Note ID pool empty!\n")); + pNotePool->free(itNewNote); + return 0; // error + } + } else { + // the parent note was apparently released already, so + // free the new note again and inform caller that it + // should drop the event + dmsg(3,("Engine: Could not assign new note as child note; Parent note is gone!\n")); + pNotePool->free(itNewNote); + return 0; // error + } + } + + dmsg(2,("Launched new note on host key %d\n", itNewNote->hostKey)); + + // copy event which caused this note + itNewNote->cause = *pNoteOnEvent; + itNewNote->eventID = pEventPool->getID(pNoteOnEvent); + + // move new note to its host key + MidiKey* pKey = &pChannel->pMIDIKeyInfo[itNewNote->hostKey]; + itNewNote.moveToEndOf(pKey->pActiveNotes); + + // assign unique note ID of this new note to the original note on event + pNoteOnEvent->Param.Note.ID = newNoteID; + + return newNoteID; // success + } + /** * Dispatch and handle all events in this audio fragment for the given * engine channel. @@ -589,7 +755,95 @@ AbstractEngineChannel* pChannel = static_cast(pEngineChannel); pChannel->ImportEvents(Samples); - // process events + // if a valid real-time instrument script is loaded, pre-process + // the event list by running the script now, since the script + // might filter events or add new ones for this cycle + if (pChannel->pScript) { + const sched_time_t fragmentEndTime = pEventGenerator->schedTimeAtCurrentFragmentEnd(); + + // resume suspended script executions been scheduled for + // this audio fragment cycle (which were suspended in a + // previous audio fragment cycle) + ProcessSuspendedScriptEvents(pChannel, fragmentEndTime); + + // spawn new script executions for the new MIDI events of + // this audio fragment cycle + // + // FIXME: it would probably be better to just schedule newly spawned script executions here and then execute them altogether with already suspended ones all at once in order of all their scheduled timing + for (RTList::Iterator itEvent = pChannel->pEvents->first(), + end = pChannel->pEvents->end(); itEvent != end; ) + { + //HACK: avoids iterator invalidation which might happen below since an instrument script might drop an event by direct raw pointer access (it would be considerable to extend the Iterator class to detect and circumvent this case by checking the "reincarnation" member variable). + RTList::Iterator itNext = itEvent; + ++itNext; + + switch (itEvent->Type) { + case Event::type_note_on: + if (pChannel->pScript->handlerNote) + ProcessEventByScript(pChannel, itEvent, pChannel->pScript->handlerNote); + break; + case Event::type_note_off: + if (pChannel->pScript->handlerRelease) + ProcessEventByScript(pChannel, itEvent, pChannel->pScript->handlerRelease); + break; + case Event::type_control_change: + case Event::type_channel_pressure: + case Event::type_pitchbend: + if (pChannel->pScript->handlerController) + ProcessEventByScript(pChannel, itEvent, pChannel->pScript->handlerController); + break; + case Event::type_note_pressure: + //TODO: ... + break; + } + + // see HACK comment above + itEvent = itNext; + } + + // this has to be run again, since the newly spawned scripts + // above may have cause suspended scripts that must be + // resumed within this same audio fragment cycle + // + // FIXME: see FIXME comment above + ProcessSuspendedScriptEvents(pChannel, fragmentEndTime); + } + + // if there are any delayed events scheduled for the current + // audio fragment cycle, then move and sort them into the main + // event list + if (!pChannel->delayedEvents.queue.isEmpty()) { + dmsg(5,("Engine: There are delayed MIDI events (total queue size: %d) ...\n", pChannel->delayedEvents.queue.size())); + const sched_time_t fragmentEndTime = pEventGenerator->schedTimeAtCurrentFragmentEnd(); + RTList::Iterator itEvent = pChannel->pEvents->first(); + while (true) { + RTList::Iterator itDelayedEventNode = + pEventGenerator->popNextScheduledEvent( + pChannel->delayedEvents.queue, + pChannel->delayedEvents.schedulerNodes, + fragmentEndTime + ); + if (!itDelayedEventNode) break; + // get the actual delayed event object and free the used scheduler node + RTList::Iterator itDelayedEvent = itDelayedEventNode->itEvent; + pChannel->delayedEvents.schedulerNodes.free(itDelayedEventNode); + if (!itDelayedEvent) { // should never happen, but just to be sure ... + dmsg(1,("Engine: Oops, invalid delayed event!\n")); + continue; + } + // skip all events on main event list which have a time + // before (or equal to) the delayed event to be inserted + for (; itEvent && itEvent->FragmentPos() <= itDelayedEvent->FragmentPos(); + ++itEvent); + // now move delayed event from delayedEvents.pList to + // the current position on the main event list + itEvent = itDelayedEvent.moveBefore(itEvent); + dmsg(5,("Engine: Inserted event of type %d into main event list (queue size: %d).\n", itEvent->Type, pChannel->delayedEvents.queue.size())); + } + dmsg(5,("Engine: End of delayed events (total queue size: %d).\n", pChannel->delayedEvents.queue.size())); + } + + // now process all events regularly { RTList::Iterator itEvent = pChannel->pEvents->first(); RTList::Iterator end = pChannel->pEvents->end(); @@ -599,18 +853,38 @@ dmsg(5,("Engine: Note on received\n")); ProcessNoteOn((EngineChannel*)itEvent->pEngineChannel, itEvent); break; + case Event::type_play_note: + dmsg(5,("Engine: Play Note received\n")); + ProcessNoteOn((EngineChannel*)itEvent->pEngineChannel, itEvent); + break; case Event::type_note_off: dmsg(5,("Engine: Note off received\n")); ProcessNoteOff((EngineChannel*)itEvent->pEngineChannel, itEvent); break; + case Event::type_stop_note: + dmsg(5,("Engine: Stop Note received\n")); + ProcessNoteOff((EngineChannel*)itEvent->pEngineChannel, itEvent); + break; case Event::type_control_change: dmsg(5,("Engine: MIDI CC received\n")); ProcessControlChange((EngineChannel*)itEvent->pEngineChannel, itEvent); break; + case Event::type_channel_pressure: + dmsg(5,("Engine: MIDI Chan. Pressure received\n")); + ProcessChannelPressure((EngineChannel*)itEvent->pEngineChannel, itEvent); + break; + case Event::type_note_pressure: + dmsg(5,("Engine: MIDI Note Pressure received\n")); + ProcessPolyphonicKeyPressure((EngineChannel*)itEvent->pEngineChannel, itEvent); + break; case Event::type_pitchbend: dmsg(5,("Engine: Pitchbend received\n")); ProcessPitchbend(static_cast(itEvent->pEngineChannel), itEvent); break; + case Event::type_note_synth_param: + dmsg(5,("Engine: Note Synth Param received\n")); + ProcessNoteSynthParam(itEvent->pEngineChannel, itEvent); + break; } } } @@ -618,22 +892,215 @@ // reset voice stealing for the next engine channel (or next audio fragment) itLastStolenVoice = VoiceIterator(); itLastStolenVoiceGlobally = VoiceIterator(); + itLastStolenNote = NoteIterator(); + itLastStolenNoteGlobally = NoteIterator(); iuiLastStolenKey = RTList::Iterator(); iuiLastStolenKeyGlobally = RTList::Iterator(); pLastStolenChannel = NULL; } /** + * Run all suspended script execution instances which are scheduled + * to be resumed for the current audio fragment cycle. + * + * @param pChannel - engine channel on which suspended events occurred + */ + void ProcessSuspendedScriptEvents(AbstractEngineChannel* pChannel, const sched_time_t fragmentEndTime) { + while (true) { + RTList::Iterator itEvent = + pEventGenerator->popNextScheduledScriptEvent( + pChannel->pScript->suspendedEvents, + *pChannel->pScript->pEvents, fragmentEndTime + ); + if (!itEvent) break; + ResumeScriptEvent(pChannel, itEvent); + } + } + + /** @brief Call instrument script's event handler for this event. + * + * Causes a new execution instance of the currently loaded real-time + * instrument script's event handler (callback) to be spawned for + * the given MIDI event. + * + * @param pChannel - engine channel on which the MIDI event occurred + * @param itEvent - MIDI event that causes this new script execution + * @param pEventHandler - script's event handler to be executed + */ + void ProcessEventByScript(AbstractEngineChannel* pChannel, RTList::Iterator& itEvent, VMEventHandler* pEventHandler) { + const int key = itEvent->Param.Note.Key; // even if this is not a note on/off event, accessing it does not mean any harm + // check if polyphonic data is passed from "note" to "release" + // script event handlers + if (pEventHandler == pChannel->pScript->handlerRelease && + pChannel->pScript->handlerNote && + pChannel->pScript->handlerNote->isPolyphonic() && + pChannel->pScript->handlerRelease->isPolyphonic() && + !pChannel->pScript->pKeyEvents[key]->isEmpty()) + { + // polyphonic variable data is used/passed from "note" to + // "release" script callback, so we have to recycle the + // original "note on" script event(s) + RTList::Iterator it = pChannel->pScript->pKeyEvents[key]->first(); + RTList::Iterator end = pChannel->pScript->pKeyEvents[key]->end(); + for (; it != end; ++it) { + ProcessScriptEvent( + pChannel, itEvent, pEventHandler, it + ); + } + } else { + // no polyphonic data is used/passed from "note" to + // "release" script callback, so just use a new fresh + // script event object + RTList::Iterator itScriptEvent = + pChannel->pScript->pEvents->allocAppend(); + ProcessScriptEvent( + pChannel, itEvent, pEventHandler, itScriptEvent + ); + } + } + + /** @brief Spawn new execution instance of an instrument script handler. + * + * Will be called to initiate a new execution of a real-time + * instrument script event right from the start of the script's + * respective handler. If script execution did not complete after + * calling this method, the respective script exeuction is then + * suspended and a call to ResumeScriptEvent() will be used next + * time to continue its execution. + * + * @param pChannel - engine channel this script is running for + * @param itEvent - event which caused execution of this script + * event handler + * @param pEventHandler - VM representation of event handler to be + * executed + * @param itScriptEvent - script event that shall be processed + */ + void ProcessScriptEvent(AbstractEngineChannel* pChannel, RTList::Iterator& itEvent, VMEventHandler* pEventHandler, RTList::Iterator& itScriptEvent) { + if (!itScriptEvent) return; // not a valid script event (i.e. because no free script event was left in the script event pool) + + // fill the list of script handlers to be executed by this event + int i = 0; + itScriptEvent->handlers[i++] = pEventHandler; // actual event handler (i.e. note, controller) + itScriptEvent->handlers[i] = NULL; // NULL termination of list + + // initialize/reset other members + itScriptEvent->cause = *itEvent; + itScriptEvent->currentHandler = 0; + itScriptEvent->executionSlices = 0; + itScriptEvent->ignoreAllWaitCalls = false; + itScriptEvent->handlerType = pEventHandler->eventHandlerType(); + // this is the native representation of the $EVENT_ID script variable + itScriptEvent->id = + (itEvent->Type == Event::type_note_on) + ? ScriptID::fromNoteID( itEvent->Param.Note.ID ) + : ScriptID::fromEventID( pEventPool->getID(itEvent) ); + + // run script handler(s) + VMExecStatus_t res = pScriptVM->exec( + pChannel->pScript->parserContext, &*itScriptEvent + ); + + // was the script suspended? + if (res & VM_EXEC_SUSPENDED) { // script was suspended ... + // in case the script was suspended, keep it on the allocated + // ScriptEvent list to be resume at the scheduled time in future, + // additionally insert it into a sorted time queue + pEventGenerator->scheduleAheadMicroSec( + pChannel->pScript->suspendedEvents, // scheduler queue + *itScriptEvent, // script event + itScriptEvent->cause.FragmentPos(), // current time of script event (basis for its next execution) + itScriptEvent->execCtx->suspensionTimeMicroseconds() // how long shall it be suspended + ); + } else { // script execution has finished without 'suspended' status ... + // if "polyphonic" variable data is passed from script's + // "note" event handler to its "release" event handler, then + // the script event must be kept and recycled for the later + // occuring "release" script event ... + if (pEventHandler == pChannel->pScript->handlerNote && + pChannel->pScript->handlerRelease && + pChannel->pScript->handlerNote->isPolyphonic() && + pChannel->pScript->handlerRelease->isPolyphonic()) + { + const int key = itEvent->Param.Note.Key; + itScriptEvent.moveToEndOf(pChannel->pScript->pKeyEvents[key & 127]); + } else { + // ... otherwise if no polyphonic data is passed and + // script's execution has finished without suspension + // status, then free the script event for a new future + // script event to be triggered from start + pChannel->pScript->pEvents->free(itScriptEvent); + } + } + } + + /** @brief Resume execution of instrument script. + * + * Will be called to resume execution of a real-time instrument + * script event which has been suspended previously. + * + * Script execution might be suspended for various reasons. Usually + * a script will be suspended if the script called the built-in + * "wait()" function, but it might also be suspended automatically + * if the script took too much execution time in an audio fragment + * cycle. So in the latter case automatic suspension is performed in + * order to avoid harm for the sampler's overall real-time + * requirements. + * + * @param pChannel - engine channel this script is running for + * @param itScriptEvent - script execution that shall be resumed + */ + void ResumeScriptEvent(AbstractEngineChannel* pChannel, RTList::Iterator& itScriptEvent) { + VMEventHandler* handler = itScriptEvent->handlers[itScriptEvent->currentHandler]; + + // run script + VMExecStatus_t res = pScriptVM->exec( + pChannel->pScript->parserContext, &*itScriptEvent + ); + + // was the script suspended? + if (res & VM_EXEC_SUSPENDED) { + // in case the script was suspended, keep it on the allocated + // ScriptEvent list to be resume at the scheduled time in future, + // additionally insert it into a sorted time queue + pEventGenerator->scheduleAheadMicroSec( + pChannel->pScript->suspendedEvents, // scheduler queue + *itScriptEvent, // script event + itScriptEvent->cause.FragmentPos(), // current time of script event (basis for its next execution) + itScriptEvent->execCtx->suspensionTimeMicroseconds() // how long shall it be suspended + ); + } else { // script execution has finished without 'suspended' status ... + // if "polyphonic" variable data is passed from script's + // "note" event handler to its "release" event handler, then + // the script event must be kept and recycled for the later + // occuring "release" script event ... + if (handler && handler == pChannel->pScript->handlerNote && + pChannel->pScript->handlerRelease && + pChannel->pScript->handlerNote->isPolyphonic() && + pChannel->pScript->handlerRelease->isPolyphonic()) + { + const int key = itScriptEvent->cause.Param.Note.Key; + itScriptEvent.moveToEndOf(pChannel->pScript->pKeyEvents[key & 127]); + } else { + // ... otherwise if no polyphonic data is passed and + // script's execution has finished without suspension + // status, then free the script event for a new future + // script event to be triggered from start + pChannel->pScript->pEvents->free(itScriptEvent); + } + } + } + + /** * Will be called by LaunchVoice() method in case there are no free * voices left. This method will select and kill one old voice for * voice stealing and postpone the note-on event until the selected * voice actually died. * - * @param pEngineChannel - engine channel on which this event occured on + * @param pEngineChannel - engine channel on which this event occurred on * @param itNoteOnEvent - key, velocity and time stamp of the event * @returns 0 on success, a value < 0 if no active voice could be picked for voice stealing */ - int StealVoice(EngineChannel* pEngineChannel, Pool::Iterator& itNoteOnEvent) { + int StealVoice(EngineChannel* pEngineChannel, Pool::Iterator& itNoteOnEvent) { if (VoiceSpawnsLeft <= 0) { dmsg(1,("Max. voice thefts per audio fragment reached (you may raise CONFIG_MAX_VOICES).\n")); return -1; @@ -641,99 +1108,124 @@ EngineChannelBase* pEngineChn = static_cast*>(pEngineChannel); - if (!pEventPool->poolIsEmpty()) { + if (pEventPool->poolIsEmpty()) { + dmsg(1,("Event pool emtpy!\n")); + return -1; + } - if(!pEngineChn->StealVoice(itNoteOnEvent, &itLastStolenVoice, &iuiLastStolenKey)) { - --VoiceSpawnsLeft; - return 0; - } + if (!pEngineChn->StealVoice(itNoteOnEvent, &itLastStolenVoice, &itLastStolenNote, &iuiLastStolenKey)) { + --VoiceSpawnsLeft; + return 0; + } - // if we couldn't steal a voice from the same engine channel then - // steal oldest voice on the oldest key from any other engine channel - // (the smaller engine channel number, the higher priority) - EngineChannelBase* pSelectedChannel; - int iChannelIndex; - VoiceIterator itSelectedVoice; - - // select engine channel - if (pLastStolenChannel) { - pSelectedChannel = pLastStolenChannel; - iChannelIndex = pSelectedChannel->iEngineIndexSelf; - } else { // pick the engine channel followed by this engine channel - iChannelIndex = (pEngineChn->iEngineIndexSelf + 1) % engineChannels.size(); - pSelectedChannel = static_cast*>(engineChannels[iChannelIndex]); - } - - // if we already stole in this fragment, try to proceed on same key - if (this->itLastStolenVoiceGlobally) { - itSelectedVoice = this->itLastStolenVoiceGlobally; - do { - ++itSelectedVoice; - } while (itSelectedVoice && !itSelectedVoice->IsStealable()); // proceed iterating if voice was created in this fragment cycle + // if we couldn't steal a voice from the same engine channel then + // steal oldest voice on the oldest key from any other engine channel + // (the smaller engine channel number, the higher priority) + EngineChannelBase* pSelectedChannel; + int iChannelIndex; + VoiceIterator itSelectedVoice; + + // select engine channel + if (pLastStolenChannel) { + pSelectedChannel = pLastStolenChannel; + iChannelIndex = pSelectedChannel->iEngineIndexSelf; + } else { // pick the engine channel followed by this engine channel + iChannelIndex = (pEngineChn->iEngineIndexSelf + 1) % engineChannels.size(); + pSelectedChannel = static_cast*>(engineChannels[iChannelIndex]); + } + + // if we already stole in this fragment, try to proceed on same note + if (this->itLastStolenVoiceGlobally) { + itSelectedVoice = this->itLastStolenVoiceGlobally; + do { + ++itSelectedVoice; + } while (itSelectedVoice && !itSelectedVoice->IsStealable()); // proceed iterating if voice was created in this fragment cycle + } + // did we find a 'stealable' voice? + if (itSelectedVoice && itSelectedVoice->IsStealable()) { + // remember which voice we stole, so we can simply proceed on next voice stealing + this->itLastStolenVoiceGlobally = itSelectedVoice; + // done + goto stealable_voice_found; + } + + // get (next) oldest note + if (this->itLastStolenNoteGlobally) { + for (NoteIterator itNote = ++this->itLastStolenNoteGlobally; + itNote; ++itNote) + { + for (itSelectedVoice = itNote->pActiveVoices->first(); itSelectedVoice; ++itSelectedVoice) { + // proceed iterating if voice was created in this audio fragment cycle + if (itSelectedVoice->IsStealable()) { + // remember which voice of which note we stole, so we can simply proceed on next voice stealing + this->itLastStolenNoteGlobally = itNote; + this->itLastStolenVoiceGlobally = itSelectedVoice; + goto stealable_voice_found; // selection succeeded + } + } } + } - #if CONFIG_DEVMODE - EngineChannel* pBegin = pSelectedChannel; // to detect endless loop - #endif // CONFIG_DEVMODE - - // did we find a 'stealable' voice? - if (itSelectedVoice && itSelectedVoice->IsStealable()) { - // remember which voice we stole, so we can simply proceed on next voice stealing - this->itLastStolenVoiceGlobally = itSelectedVoice; - } else while (true) { // iterate through engine channels - // get (next) oldest key - RTList::Iterator iuiSelectedKey = (this->iuiLastStolenKeyGlobally) ? ++this->iuiLastStolenKeyGlobally : pSelectedChannel->pActiveKeys->first(); - this->iuiLastStolenKeyGlobally = RTList::Iterator(); // to prevent endless loop (see line above) - while (iuiSelectedKey) { - MidiKey* pSelectedKey = &pSelectedChannel->pMIDIKeyInfo[*iuiSelectedKey]; - itSelectedVoice = pSelectedKey->pActiveVoices->first(); + #if CONFIG_DEVMODE + EngineChannel* pBegin = pSelectedChannel; // to detect endless loop + #endif // CONFIG_DEVMODE + + while (true) { // iterate through engine channels + // get (next) oldest key + RTList::Iterator iuiSelectedKey = (this->iuiLastStolenKeyGlobally) ? ++this->iuiLastStolenKeyGlobally : pSelectedChannel->pActiveKeys->first(); + this->iuiLastStolenKeyGlobally = RTList::Iterator(); // to prevent endless loop (see line above) + while (iuiSelectedKey) { + MidiKey* pSelectedKey = &pSelectedChannel->pMIDIKeyInfo[*iuiSelectedKey]; + + for (NoteIterator itNote = pSelectedKey->pActiveNotes->first(), + itNotesEnd = pSelectedKey->pActiveNotes->end(); + itNote != itNotesEnd; ++itNote) + { + itSelectedVoice = itNote->pActiveVoices->first(); // proceed iterating if voice was created in this fragment cycle while (itSelectedVoice && !itSelectedVoice->IsStealable()) ++itSelectedVoice; // found a "stealable" voice ? if (itSelectedVoice && itSelectedVoice->IsStealable()) { - // remember which voice on which key on which engine channel we stole, so we can simply proceed on next voice stealing + // remember which voice of which note on which key on which engine channel we stole, so we can simply proceed on next voice stealing this->iuiLastStolenKeyGlobally = iuiSelectedKey; + this->itLastStolenNoteGlobally = itNote; this->itLastStolenVoiceGlobally = itSelectedVoice; this->pLastStolenChannel = pSelectedChannel; goto stealable_voice_found; // selection succeeded } - ++iuiSelectedKey; // get next key on current engine channel } - // get next engine channel - iChannelIndex = (iChannelIndex + 1) % engineChannels.size(); - pSelectedChannel = static_cast*>(engineChannels[iChannelIndex]); - - #if CONFIG_DEVMODE - if (pSelectedChannel == pBegin) { - dmsg(1,("FATAL ERROR: voice stealing endless loop!\n")); - dmsg(1,("VoiceSpawnsLeft=%d.\n", VoiceSpawnsLeft)); - dmsg(1,("Exiting.\n")); - exit(-1); - } - #endif // CONFIG_DEVMODE + ++iuiSelectedKey; // get next key on current engine channel } - - // jump point if a 'stealable' voice was found - stealable_voice_found: + // get next engine channel + iChannelIndex = (iChannelIndex + 1) % engineChannels.size(); + pSelectedChannel = static_cast*>(engineChannels[iChannelIndex]); #if CONFIG_DEVMODE - if (!itSelectedVoice->IsActive()) { - dmsg(1,("EngineBase: ERROR, tried to steal a voice which was not active !!!\n")); - return -1; + if (pSelectedChannel == pBegin) { + dmsg(1,("FATAL ERROR: voice stealing endless loop!\n")); + dmsg(1,("VoiceSpawnsLeft=%d.\n", VoiceSpawnsLeft)); + dmsg(1,("Exiting.\n")); + exit(-1); } #endif // CONFIG_DEVMODE + } - // now kill the selected voice - itSelectedVoice->Kill(itNoteOnEvent); - - --VoiceSpawnsLeft; + // jump point if a 'stealable' voice was found + stealable_voice_found: - return 0; // success - } - else { - dmsg(1,("Event pool emtpy!\n")); + #if CONFIG_DEVMODE + if (!itSelectedVoice->IsActive()) { + dmsg(1,("EngineBase: ERROR, tried to steal a voice which was not active !!!\n")); return -1; } + #endif // CONFIG_DEVMODE + + // now kill the selected voice + itSelectedVoice->Kill(itNoteOnEvent); + + --VoiceSpawnsLeft; + + return 0; // success } void HandleInstrumentChanges() { @@ -755,9 +1247,28 @@ dmsg(5,("Engine: instrument change command received\n")); cmd.bChangeInstrument = false; pEngineChannel->pInstrument = cmd.pInstrument; + pEngineChannel->pScript = + cmd.pScript->bHasValidScript ? cmd.pScript : NULL; instrumentChanged = true; pEngineChannel->MarkAllActiveVoicesAsOrphans(); + + // the script's "init" event handler is only executed + // once (when the script is loaded or reloaded) + if (pEngineChannel->pScript && pEngineChannel->pScript->handlerInit) { + RTList::Iterator itScriptEvent = + pEngineChannel->pScript->pEvents->allocAppend(); + + itScriptEvent->cause.pEngineChannel = pEngineChannel; + itScriptEvent->handlers[0] = pEngineChannel->pScript->handlerInit; + itScriptEvent->handlers[1] = NULL; + + VMExecStatus_t res = pScriptVM->exec( + pEngineChannel->pScript->parserContext, &*itScriptEvent + ); + + pEngineChannel->pScript->pEvents->free(itScriptEvent); + } } } @@ -765,12 +1276,6 @@ //TODO: this is a lazy solution ATM and not safe in case somebody is currently editing the instrument we're currently switching to (we should store all suspended regions on instrument manager side and when switching to another instrument copy that list to the engine's local list of suspensions ResetSuspendedRegions(); } - - for (int i = 0; i < engineChannels.size(); i++) { - EngineChannelBase* channel = - static_cast*>(engineChannels[i]); - channel->InstrumentChangeCommandReader.Unlock(); - } } /** @@ -814,9 +1319,23 @@ EngineChannelBase* pEngineChannel = static_cast*>(itVoiceStealEvent->pEngineChannel);; if (!pEngineChannel->pInstrument) continue; // ignore if no instrument loaded + PoolVoiceIterator itNewVoice = LaunchVoice(pEngineChannel, itVoiceStealEvent, itVoiceStealEvent->Param.Note.Layer, itVoiceStealEvent->Param.Note.ReleaseTrigger, false, false); if (itNewVoice) { + // usually there should already be a new Note object + NoteIterator itNote = GetNotePool()->fromID(itVoiceStealEvent->Param.Note.ID); + if (!itNote) { // should not happen, but just to be sure ... + const note_id_t noteID = LaunchNewNote(pEngineChannel, &*itVoiceStealEvent); + if (!noteID) { + dmsg(1,("Engine: Voice stealing failed; No Note object and Note pool empty!\n")); + continue; + } + itNote = GetNotePool()->fromID(noteID); + } + // move voice from whereever it was, to the new note's list of active voices + itNewVoice = itNewVoice.moveToEndOf(itNote->pActiveVoices); + // render audio of this new voice for the first time itNewVoice->Render(Samples); if (itNewVoice->IsActive()) { // still active *(pEngineChannel->pRegionsInUse->allocAppend()) = itNewVoice->GetRegion(); @@ -854,9 +1373,18 @@ pChannel->FreeAllInactiveKyes(); // empty the engine channel's own event lists - pChannel->ClearEventLists(); + // (only events of the current audio fragment cycle) + pChannel->ClearEventListsOfCurrentFragment(); } + /** + * Process MIDI control change events with hard coded behavior, + * that is controllers whose behavior is defined independently + * of the actual sampler engine type and instrument. + * + * @param pEngineChannel - engine channel on which the MIDI CC event was received + * @param itControlChangeEvent - the actual MIDI CC event + */ void ProcessHardcodedControllers ( EngineChannel* pEngineChannel, Pool::Iterator& itControlChangeEvent @@ -869,18 +1397,65 @@ pChannel->PortamentoTime = (float) itControlChangeEvent->Param.CC.Value / 127.0f * (float) CONFIG_PORTAMENTO_TIME_MAX + (float) CONFIG_PORTAMENTO_TIME_MIN; break; } - case 6: { // data entry (currently only used for RPN controllers) - if (pChannel->GetMidiRpnController() == 2) { // coarse tuning in half tones - int transpose = (int) itControlChangeEvent->Param.CC.Value - 64; - // limit to +- two octaves for now - transpose = RTMath::Min(transpose, 24); - transpose = RTMath::Max(transpose, -24); - pChannel->GlobalTranspose = transpose; - // workaround, so we won't have hanging notes - pChannel->ReleaseAllVoices(itControlChangeEvent); + case 6: { // data entry (currently only used for RPN and NRPN controllers) + //dmsg(1,("DATA ENTRY %d\n", itControlChangeEvent->Param.CC.Value)); + if (pChannel->GetMidiRpnController() >= 0) { // RPN controller number was sent previously ... + dmsg(4,("Guess it's an RPN ...\n")); + if (pChannel->GetMidiRpnController() == 2) { // coarse tuning in half tones + int transpose = (int) itControlChangeEvent->Param.CC.Value - 64; + // limit to +- two octaves for now + transpose = RTMath::Min(transpose, 24); + transpose = RTMath::Max(transpose, -24); + pChannel->GlobalTranspose = transpose; + // workaround, so we won't have hanging notes + pChannel->ReleaseAllVoices(itControlChangeEvent); + } + // to prevent other MIDI CC #6 messages to be misenterpreted as RPN controller data + pChannel->ResetMidiRpnController(); + } else if (pChannel->GetMidiNrpnController() >= 0) { // NRPN controller number was sent previously ... + dmsg(4,("Guess it's an NRPN ...\n")); + const int NrpnCtrlMSB = pChannel->GetMidiNrpnController() >> 8; + const int NrpnCtrlLSB = pChannel->GetMidiNrpnController() & 0xff; + dmsg(4,("NRPN MSB=%d LSB=%d Data=%d\n", NrpnCtrlMSB, NrpnCtrlLSB, itControlChangeEvent->Param.CC.Value)); + switch (NrpnCtrlMSB) { + case 0x1a: { // volume level of note (Roland GS NRPN) + const uint note = NrpnCtrlLSB; + const uint vol = itControlChangeEvent->Param.CC.Value; + dmsg(4,("Note Volume NRPN received (note=%d,vol=%d).\n", note, vol)); + if (note < 128 && vol < 128) + pChannel->pMIDIKeyInfo[note].Volume = VolumeCurve[vol]; + break; + } + case 0x1c: { // panpot of note (Roland GS NRPN) + const uint note = NrpnCtrlLSB; + const uint pan = itControlChangeEvent->Param.CC.Value; + dmsg(4,("Note Pan NRPN received (note=%d,pan=%d).\n", note, pan)); + if (note < 128 && pan < 128) { + pChannel->pMIDIKeyInfo[note].PanLeft = PanCurve[128 - pan]; + pChannel->pMIDIKeyInfo[note].PanRight = PanCurve[pan]; + } + break; + } + case 0x1d: { // reverb send of note (Roland GS NRPN) + const uint note = NrpnCtrlLSB; + const float reverb = float(itControlChangeEvent->Param.CC.Value) / 127.0f; + dmsg(4,("Note Reverb Send NRPN received (note=%d,send=%f).\n", note, reverb)); + if (note < 128) + pChannel->pMIDIKeyInfo[note].ReverbSend = reverb; + break; + } + case 0x1e: { // chorus send of note (Roland GS NRPN) + const uint note = NrpnCtrlLSB; + const float chorus = float(itControlChangeEvent->Param.CC.Value) / 127.0f; + dmsg(4,("Note Chorus Send NRPN received (note=%d,send=%f).\n", note, chorus)); + if (note < 128) + pChannel->pMIDIKeyInfo[note].ChorusSend = chorus; + break; + } + } + // to prevent other MIDI CC #6 messages to be misenterpreted as NRPN controller data + pChannel->ResetMidiNrpnController(); } - // to avoid other MIDI CC #6 messages to be misenterpreted as RPN controller data - pChannel->ResetMidiRpnController(); break; } case 7: { // volume @@ -891,8 +1466,6 @@ } case 10: { // panpot //TODO: not sample accurate yet - pChannel->GlobalPanLeft = PanCurve[128 - itControlChangeEvent->Param.CC.Value]; - pChannel->GlobalPanRight = PanCurve[itControlChangeEvent->Param.CC.Value]; pChannel->iLastPanRequest = itControlChangeEvent->Param.CC.Value; break; } @@ -969,11 +1542,23 @@ } break; } + case 98: { // NRPN controller LSB + dmsg(4,("NRPN LSB %d\n", itControlChangeEvent->Param.CC.Value)); + pEngineChannel->SetMidiNrpnControllerLsb(itControlChangeEvent->Param.CC.Value); + break; + } + case 99: { // NRPN controller MSB + dmsg(4,("NRPN MSB %d\n", itControlChangeEvent->Param.CC.Value)); + pEngineChannel->SetMidiNrpnControllerMsb(itControlChangeEvent->Param.CC.Value); + break; + } case 100: { // RPN controller LSB + dmsg(4,("RPN LSB %d\n", itControlChangeEvent->Param.CC.Value)); pEngineChannel->SetMidiRpnControllerLsb(itControlChangeEvent->Param.CC.Value); break; } case 101: { // RPN controller MSB + dmsg(4,("RPN MSB %d\n", itControlChangeEvent->Param.CC.Value)); pEngineChannel->SetMidiRpnControllerMsb(itControlChangeEvent->Param.CC.Value); break; } @@ -1015,33 +1600,40 @@ /** * Assigns and triggers a new voice for the respective MIDI key. * - * @param pEngineChannel - engine channel on which this event occured on + * @param pEngineChannel - engine channel on which this event occurred on * @param itNoteOnEvent - key, velocity and time stamp of the event */ virtual void ProcessNoteOn(EngineChannel* pEngineChannel, Pool::Iterator& itNoteOnEvent) { EngineChannelBase* pChannel = static_cast*>(pEngineChannel); - //HACK: we should better add the transpose value only to the most mandatory places (like for retrieving the region and calculating the tuning), because otherwise voices will unintendedly survive when changing transpose while playing - int k = itNoteOnEvent->Param.Note.Key + pChannel->GlobalTranspose; - if (k < 0 || k > 127) return; //ignore keys outside the key range - - itNoteOnEvent->Param.Note.Key += pChannel->GlobalTranspose; - int vel = itNoteOnEvent->Param.Note.Velocity; - const int key = itNoteOnEvent->Param.Note.Key; + const int vel = itNoteOnEvent->Param.Note.Velocity; + if (key < 0 || key > 127) return; // ignore event, key outside allowed key range + MidiKey* pKey = &pChannel->pMIDIKeyInfo[key]; - pChannel->listeners.PreProcessNoteOn(key, vel); + // There are real MIDI note-on events (Event::type_note_on) and + // programmatically spawned notes (Event::type_play_note). We have + // to distinguish between them, since certain processing below + // must only be done on real MIDI note-on events (i.e. for + // correctly updating which MIDI keys are currently pressed down). + const bool isRealMIDINoteOnEvent = itNoteOnEvent->Type == Event::type_note_on; + + if (isRealMIDINoteOnEvent) + pChannel->listeners.PreProcessNoteOn(key, vel); + #if !CONFIG_PROCESS_MUTED_CHANNELS if (pEngineChannel->GetMute()) { // skip if sampler channel is muted - pChannel->listeners.PostProcessNoteOn(key, vel); + if (isRealMIDINoteOnEvent) + pChannel->listeners.PostProcessNoteOn(key, vel); return; } #endif if (!pChannel->pInstrument) { - pChannel->listeners.PostProcessNoteOn(key, vel); + if (isRealMIDINoteOnEvent) + pChannel->listeners.PostProcessNoteOn(key, vel); return; // ignore if no instrument loaded } @@ -1049,7 +1641,7 @@ RTList::Iterator itNoteOnEventOnKeyList = itNoteOnEvent.moveToEndOf(pKey->pEvents); // if Solo Mode then kill all already active voices - if (pChannel->SoloMode) { + if (pChannel->SoloMode && isRealMIDINoteOnEvent) { Pool::Iterator itYoungestKey = pChannel->pActiveKeys->last(); if (itYoungestKey) { const int iYoungestKey = *itYoungestKey; @@ -1057,15 +1649,20 @@ if (pOtherKey->Active) { // get final portamento position of currently active voice if (pChannel->PortamentoMode) { - VoiceIterator itVoice = pOtherKey->pActiveVoices->last(); - if (itVoice) itVoice->UpdatePortamentoPos(itNoteOnEventOnKeyList); + NoteIterator itNote = pOtherKey->pActiveNotes->last(); + if (itNote) { + VoiceIterator itVoice = itNote->pActiveVoices->last(); + if (itVoice) itVoice->UpdatePortamentoPos(itNoteOnEventOnKeyList); + } } // kill all voices on the (other) key - VoiceIterator itVoiceToBeKilled = pOtherKey->pActiveVoices->first(); - VoiceIterator end = pOtherKey->pActiveVoices->end(); - for (; itVoiceToBeKilled != end; ++itVoiceToBeKilled) { - if (!(itVoiceToBeKilled->Type & Voice::type_release_trigger)) - itVoiceToBeKilled->Kill(itNoteOnEventOnKeyList); + for (NoteIterator itNote = pOtherKey->pActiveNotes->first(); itNote; ++itNote) { + VoiceIterator itVoiceToBeKilled = itNote->pActiveVoices->first(); + VoiceIterator end = itNote->pActiveVoices->end(); + for (; itVoiceToBeKilled != end; ++itVoiceToBeKilled) { + if (!(itVoiceToBeKilled->Type & Voice::type_release_trigger)) + itVoiceToBeKilled->Kill(itNoteOnEventOnKeyList); + } } } } @@ -1073,18 +1670,21 @@ pChannel->SoloKey = key; } - pChannel->ProcessKeySwitchChange(key); + if (isRealMIDINoteOnEvent) { + pChannel->ProcessKeySwitchChange(key); - pKey->KeyPressed = true; // the MIDI key was now pressed down - pKey->Velocity = itNoteOnEventOnKeyList->Param.Note.Velocity; - pKey->NoteOnTime = FrameTime + itNoteOnEventOnKeyList->FragmentPos(); // will be used to calculate note length + pKey->KeyPressed = true; // the MIDI key was now pressed down + pChannel->KeyDown[key] = true; // just used as built-in %KEY_DOWN script variable + pKey->Velocity = itNoteOnEventOnKeyList->Param.Note.Velocity; + pKey->NoteOnTime = FrameTime + itNoteOnEventOnKeyList->FragmentPos(); // will be used to calculate note length + } // cancel release process of voices on this key if needed - if (pKey->Active && !pChannel->SustainPedal) { + if (pKey->Active && !pChannel->SustainPedal && isRealMIDINoteOnEvent) { RTList::Iterator itCancelReleaseEvent = pKey->pEvents->allocAppend(); if (itCancelReleaseEvent) { *itCancelReleaseEvent = *itNoteOnEventOnKeyList; // copy event - itCancelReleaseEvent->Type = Event::type_cancel_release; // transform event type + itCancelReleaseEvent->Type = Event::type_cancel_release_key; // transform event type } else dmsg(1,("Event pool emtpy!\n")); } @@ -1095,12 +1695,17 @@ if (!pKey->Active && !pKey->VoiceTheftsQueued) pKey->pEvents->free(itNoteOnEventOnKeyList); - if (!pChannel->SoloMode || pChannel->PortamentoPos < 0.0f) pChannel->PortamentoPos = (float) key; + if (isRealMIDINoteOnEvent && (!pChannel->SoloMode || pChannel->PortamentoPos < 0.0f)) + pChannel->PortamentoPos = (float) key; + + //NOTE: Hmm, I guess its a matter of taste whether round robin should be advanced only on real MIDI note-on events, isn't it? if (pKey->pRoundRobinIndex) { (*pKey->pRoundRobinIndex)++; // counter specific for the key or region pChannel->RoundRobinIndex++; // common counter for the channel } - pChannel->listeners.PostProcessNoteOn(key, vel); + + if (isRealMIDINoteOnEvent) + pChannel->listeners.PostProcessNoteOn(key, vel); } /** @@ -1126,96 +1731,123 @@ * sustain pedal will be released or voice turned inactive by itself (e.g. * due to completion of sample playback). * - * @param pEngineChannel - engine channel on which this event occured on + * @param pEngineChannel - engine channel on which this event occurred on * @param itNoteOffEvent - key, velocity and time stamp of the event */ virtual void ProcessNoteOff(EngineChannel* pEngineChannel, Pool::Iterator& itNoteOffEvent) { EngineChannelBase* pChannel = static_cast*>(pEngineChannel); - int k = itNoteOffEvent->Param.Note.Key + pChannel->GlobalTranspose; - if (k < 0 || k > 127) return; //ignore keys outside the key range - - //HACK: we should better add the transpose value only to the most mandatory places (like for retrieving the region and calculating the tuning), because otherwise voices will unintendedly survive when changing transpose while playing - itNoteOffEvent->Param.Note.Key += pChannel->GlobalTranspose; - int vel = itNoteOffEvent->Param.Note.Velocity; - const int iKey = itNoteOffEvent->Param.Note.Key; + const int vel = itNoteOffEvent->Param.Note.Velocity; + if (iKey < 0 || iKey > 127) return; // ignore event, key outside allowed key range + MidiKey* pKey = &pChannel->pMIDIKeyInfo[iKey]; - pChannel->listeners.PreProcessNoteOff(iKey, vel); + // There are real MIDI note-off events (Event::type_note_off) and + // programmatically spawned notes (Event::type_stop_note). We have + // to distinguish between them, since certain processing below + // must only be done on real MIDI note-off events (i.e. for + // correctly updating which MIDI keys are currently pressed down), + // plus a stop-note event just releases voices of one particular + // note, whereas a note-off event releases all voices on a + // particular MIDI key instead. + const bool isRealMIDINoteOffEvent = itNoteOffEvent->Type == Event::type_note_off; + + if (isRealMIDINoteOffEvent) + pChannel->listeners.PreProcessNoteOff(iKey, vel); #if !CONFIG_PROCESS_MUTED_CHANNELS if (pEngineChannel->GetMute()) { // skip if sampler channel is muted - pChannel->listeners.PostProcessNoteOff(iKey, vel); + if (isRealMIDINoteOffEvent) + pChannel->listeners.PostProcessNoteOff(iKey, vel); return; } #endif - pKey->KeyPressed = false; // the MIDI key was now released + if (isRealMIDINoteOffEvent) { + pKey->KeyPressed = false; // the MIDI key was now released + pChannel->KeyDown[iKey] = false; // just used as built-in %KEY_DOWN script variable + } // move event to the key's own event list RTList::Iterator itNoteOffEventOnKeyList = itNoteOffEvent.moveToEndOf(pKey->pEvents); - bool bShouldRelease = pKey->Active && pChannel->ShouldReleaseVoice(itNoteOffEventOnKeyList->Param.Note.Key); + if (isRealMIDINoteOffEvent) { + bool bShouldRelease = pKey->Active && pChannel->ShouldReleaseVoice(itNoteOffEventOnKeyList->Param.Note.Key); - // in case Solo Mode is enabled, kill all voices on this key and respawn a voice on the highest pressed key (if any) - if (pChannel->SoloMode && pChannel->pInstrument) { //TODO: this feels like too much code just for handling solo mode :P - bool bOtherKeysPressed = false; - if (iKey == pChannel->SoloKey) { - pChannel->SoloKey = -1; - // if there's still a key pressed down, respawn a voice (group) on the highest key - for (int i = 127; i > 0; i--) { - MidiKey* pOtherKey = &pChannel->pMIDIKeyInfo[i]; - if (pOtherKey->KeyPressed) { - bOtherKeysPressed = true; - // make the other key the new 'currently active solo key' - pChannel->SoloKey = i; - // get final portamento position of currently active voice - if (pChannel->PortamentoMode) { - VoiceIterator itVoice = pKey->pActiveVoices->first(); - if (itVoice) itVoice->UpdatePortamentoPos(itNoteOffEventOnKeyList); - } - // create a pseudo note on event - RTList::Iterator itPseudoNoteOnEvent = pOtherKey->pEvents->allocAppend(); - if (itPseudoNoteOnEvent) { - // copy event - *itPseudoNoteOnEvent = *itNoteOffEventOnKeyList; - // transform event to a note on event - itPseudoNoteOnEvent->Type = Event::type_note_on; - itPseudoNoteOnEvent->Param.Note.Key = i; - itPseudoNoteOnEvent->Param.Note.Velocity = pOtherKey->Velocity; - // allocate and trigger new voice(s) for the other key - TriggerNewVoices(pChannel, itPseudoNoteOnEvent, false); - // if neither a voice was spawned or postponed then remove note on event from key again - if (!pOtherKey->Active && !pOtherKey->VoiceTheftsQueued) - pOtherKey->pEvents->free(itPseudoNoteOnEvent); + // in case Solo Mode is enabled, kill all voices on this key and respawn a voice on the highest pressed key (if any) + if (pChannel->SoloMode && pChannel->pInstrument) { //TODO: this feels like too much code just for handling solo mode :P + bool bOtherKeysPressed = false; + if (iKey == pChannel->SoloKey) { + pChannel->SoloKey = -1; + // if there's still a key pressed down, respawn a voice (group) on the highest key + for (int i = 127; i > 0; i--) { + MidiKey* pOtherKey = &pChannel->pMIDIKeyInfo[i]; + if (pOtherKey->KeyPressed) { + bOtherKeysPressed = true; + // make the other key the new 'currently active solo key' + pChannel->SoloKey = i; + // get final portamento position of currently active voice + if (pChannel->PortamentoMode) { + NoteIterator itNote = pKey->pActiveNotes->first(); + VoiceIterator itVoice = itNote->pActiveVoices->first(); + if (itVoice) itVoice->UpdatePortamentoPos(itNoteOffEventOnKeyList); + } + // create a pseudo note on event + RTList::Iterator itPseudoNoteOnEvent = pOtherKey->pEvents->allocAppend(); + if (itPseudoNoteOnEvent) { + // copy event + *itPseudoNoteOnEvent = *itNoteOffEventOnKeyList; + // transform event to a note on event + itPseudoNoteOnEvent->Type = Event::type_note_on; //FIXME: should probably use Event::type_play_note instead (to avoid i.e. hanging notes) + itPseudoNoteOnEvent->Param.Note.Key = i; + itPseudoNoteOnEvent->Param.Note.Velocity = pOtherKey->Velocity; + // assign a new note to this note-on event + if (LaunchNewNote(pChannel, &*itPseudoNoteOnEvent)) { + // allocate and trigger new voice(s) for the other key + TriggerNewVoices(pChannel, itPseudoNoteOnEvent, false); + } + // if neither a voice was spawned or postponed then remove note on event from key again + if (!pOtherKey->Active && !pOtherKey->VoiceTheftsQueued) + pOtherKey->pEvents->free(itPseudoNoteOnEvent); - } else dmsg(1,("Could not respawn voice, no free event left\n")); - break; // done + } else dmsg(1,("Could not respawn voice, no free event left\n")); + break; // done + } } } - } - if (bOtherKeysPressed) { - if (pKey->Active) { // kill all voices on this key - bShouldRelease = false; // no need to release, as we kill it here - VoiceIterator itVoiceToBeKilled = pKey->pActiveVoices->first(); - VoiceIterator end = pKey->pActiveVoices->end(); - for (; itVoiceToBeKilled != end; ++itVoiceToBeKilled) { - if (!(itVoiceToBeKilled->Type & Voice::type_release_trigger)) - itVoiceToBeKilled->Kill(itNoteOffEventOnKeyList); + if (bOtherKeysPressed) { + if (pKey->Active) { // kill all voices on this key + bShouldRelease = false; // no need to release, as we kill it here + for (NoteIterator itNote = pKey->pActiveNotes->first(); itNote; ++itNote) { + VoiceIterator itVoiceToBeKilled = itNote->pActiveVoices->first(); + VoiceIterator end = itNote->pActiveVoices->end(); + for (; itVoiceToBeKilled != end; ++itVoiceToBeKilled) { + if (!(itVoiceToBeKilled->Type & Voice::type_release_trigger)) + itVoiceToBeKilled->Kill(itNoteOffEventOnKeyList); + } + } } - } - } else pChannel->PortamentoPos = -1.0f; - } - - // if no solo mode (the usual case) or if solo mode and no other key pressed, then release voices on this key if needed - if (bShouldRelease) { - itNoteOffEventOnKeyList->Type = Event::type_release; // transform event type + } else pChannel->PortamentoPos = -1.0f; + } - // spawn release triggered voice(s) if needed - if (pKey->ReleaseTrigger && pChannel->pInstrument) { - TriggerReleaseVoices(pChannel, itNoteOffEventOnKeyList); - pKey->ReleaseTrigger = false; + // if no solo mode (the usual case) or if solo mode and no other key pressed, then release voices on this key if needed + if (bShouldRelease) { + itNoteOffEventOnKeyList->Type = Event::type_release_key; // transform event type + // spawn release triggered voice(s) if needed + ProcessReleaseTrigger(pChannel, itNoteOffEventOnKeyList, pKey); + } + } else if (itNoteOffEventOnKeyList->Type == Event::type_stop_note) { + // This programmatically caused event is caused by a call to + // the built-in instrument script function note_off(). In + // contrast to a real MIDI note-off event the stop-note + // event just intends to release voices of one particular note. + NoteBase* pNote = pChannel->pEngine->NoteByID( itNoteOffEventOnKeyList->Param.Note.ID ); + if (pNote) { // the requested note is still alive ... + itNoteOffEventOnKeyList->Type = Event::type_release_note; // transform event type + } else { // note is dead and gone .. + pKey->pEvents->free(itNoteOffEventOnKeyList); // remove stop-note event from key again + return; // prevent event to be removed a 2nd time below } } @@ -1223,7 +1855,119 @@ if (!pKey->Active && !pKey->VoiceTheftsQueued) pKey->pEvents->free(itNoteOffEventOnKeyList); - pChannel->listeners.PostProcessNoteOff(iKey, vel); + if (isRealMIDINoteOffEvent) + pChannel->listeners.PostProcessNoteOff(iKey, vel); + } + + /** + * Called on sustain pedal up events to check and if required, + * launch release trigger voices on the respective active key. + * + * @param pEngineChannel - engine channel on which this event occurred on + * @param itEvent - release trigger event (contains note number) + */ + virtual void ProcessReleaseTrigger(EngineChannel* pEngineChannel, RTList::Iterator& itEvent) OVERRIDE { + EngineChannelBase* pChannel = static_cast*>(pEngineChannel); + + const int iKey = itEvent->Param.Note.Key; + if (iKey < 0 || iKey > 127) return; // ignore event, key outside allowed key range + + MidiKey* pKey = &pChannel->pMIDIKeyInfo[iKey]; + + ProcessReleaseTrigger(pChannel, itEvent, pKey); + } + + /** + * Called on note-off and sustain pedal up events to check and if + * required, launch release trigger voices on the respective active + * key. + * + * @param pEngineChannel - engine channel on which this event occurred on + * @param itEvent - note off event / release trigger event + * @param pKey - key on which the release trigger voices shall be spawned + */ + inline void ProcessReleaseTrigger(EngineChannelBase* pChannel, RTList::Iterator& itEvent, MidiKey* pKey) { + // spawn release triggered voice(s) if needed + if (pKey->ReleaseTrigger && pChannel->pInstrument) { + // assign a new note to this release event + if (LaunchNewNote(pChannel, &*itEvent)) { + // allocate and trigger new release voice(s) + TriggerReleaseVoices(pChannel, itEvent); + } + pKey->ReleaseTrigger = false; + } + } + + /** + * Called on note synthesis parameter change events. These are + * internal events caused by calling built-in real-time instrument + * script functions like change_vol(), change_pitch(), etc. + * + * This method performs two tasks: + * + * - It converts the event's relative values changes (Deltas) to + * the respective final new synthesis parameter value (AbsValue), + * for that particular moment of the event that is. + * + * - It moves the individual events to the Note's own event list + * (or actually to the event list of the MIDI key), so that + * voices can process those events sample accurately. + * + * @param pEngineChannel - engine channel on which this event occurred on + * @param itEvent - note synthesis parameter change event + */ + virtual void ProcessNoteSynthParam(EngineChannel* pEngineChannel, RTList::Iterator& itEvent) { + EngineChannelBase* pChannel = static_cast*>(pEngineChannel); + + NoteBase* pNote = pChannel->pEngine->NoteByID( itEvent->Param.NoteSynthParam.NoteID ); + if (!pNote || pNote->hostKey < 0 || pNote->hostKey >= 128) return; + + const bool& relative = itEvent->Param.NoteSynthParam.Relative; + + switch (itEvent->Param.NoteSynthParam.Type) { + case Event::synth_param_volume: + if (relative) + pNote->Override.Volume *= itEvent->Param.NoteSynthParam.Delta; + else + pNote->Override.Volume = itEvent->Param.NoteSynthParam.Delta; + itEvent->Param.NoteSynthParam.AbsValue = pNote->Override.Volume; + break; + case Event::synth_param_pitch: + if (relative) + pNote->Override.Pitch *= itEvent->Param.NoteSynthParam.Delta; + else + pNote->Override.Pitch = itEvent->Param.NoteSynthParam.Delta; + itEvent->Param.NoteSynthParam.AbsValue = pNote->Override.Pitch; + break; + case Event::synth_param_pan: + if (relative) { + pNote->Override.Pan = RTMath::RelativeSummedAvg(pNote->Override.Pan, itEvent->Param.NoteSynthParam.Delta, ++pNote->Override.PanSources); + } else { + pNote->Override.Pan = itEvent->Param.NoteSynthParam.Delta; + pNote->Override.PanSources = 1; // only relevant on subsequent change_pan() instrument script calls on same note with 'relative' argument being set + } + itEvent->Param.NoteSynthParam.AbsValue = pNote->Override.Pan; + break; + case Event::synth_param_cutoff: + pNote->Override.Cutoff = itEvent->Param.NoteSynthParam.AbsValue = itEvent->Param.NoteSynthParam.Delta; + break; + case Event::synth_param_resonance: + pNote->Override.Resonance = itEvent->Param.NoteSynthParam.AbsValue = itEvent->Param.NoteSynthParam.Delta; + break; + case Event::synth_param_attack: + pNote->Override.Attack = itEvent->Param.NoteSynthParam.AbsValue = itEvent->Param.NoteSynthParam.Delta; + break; + case Event::synth_param_decay: + pNote->Override.Decay = itEvent->Param.NoteSynthParam.AbsValue = itEvent->Param.NoteSynthParam.Delta; + break; + case Event::synth_param_release: + pNote->Override.Release = itEvent->Param.NoteSynthParam.AbsValue = itEvent->Param.NoteSynthParam.Delta; + break; + } + + // move note parameter event to its MIDI key + MidiKey* pKey = &pChannel->pMIDIKeyInfo[pNote->hostKey]; + itEvent.moveToEndOf(pKey->pEvents); } /** @@ -1231,7 +1975,7 @@ * control and status variables. This method is protected by a mutex. */ virtual void ResetInternal() { - ResetInternalMutex.Lock(); + LockGuard lock(ResetInternalMutex); // make sure that the engine does not get any sysex messages // while it's reseting @@ -1243,16 +1987,35 @@ pVoiceStealingQueue->clear(); itLastStolenVoice = VoiceIterator(); itLastStolenVoiceGlobally = VoiceIterator(); + itLastStolenNote = NoteIterator(); + itLastStolenNoteGlobally = NoteIterator(); iuiLastStolenKey = RTList::Iterator(); iuiLastStolenKeyGlobally = RTList::Iterator(); pLastStolenChannel = NULL; + // reset all notes + pNotePool->clear(); + for (NoteIterator itNote = pNotePool->allocAppend(); itNote; + itNote = pNotePool->allocAppend()) + { + itNote->reset(); + } + pNotePool->clear(); + // reset all voices + pVoicePool->clear(); for (VoiceIterator iterVoice = pVoicePool->allocAppend(); iterVoice == pVoicePool->last(); iterVoice = pVoicePool->allocAppend()) { iterVoice->Reset(); } pVoicePool->clear(); + // reset all engine channels + for (int i = 0; i < engineChannels.size(); i++) { + AbstractEngineChannel* pEngineChannel = + static_cast(engineChannels[i]); + pEngineChannel->ResetInternal(false/*don't reset engine*/); + } + // reset disk thread if (pDiskThread) pDiskThread->Reset(); @@ -1260,7 +2023,6 @@ pEventQueue->init(); pSysexBuffer->init(); if (sysexDisabled) MidiInputPort::AddSysexListener(this); - ResetInternalMutex.Unlock(); } /** @@ -1282,7 +2044,7 @@ * called by the ProcessNoteOn() method and by the voices itself * (e.g. to spawn further voices on the same key for layered sounds). * - * @param pEngineChannel - engine channel on which this event occured on + * @param pEngineChannel - engine channel on which this event occurred on * @param itNoteOnEvent - key, velocity and time stamp of the event * @param iLayer - layer index for the new voice (optional - only * in case of layered sounds of course) @@ -1325,7 +2087,7 @@ // launch the new voice if (itNewVoice->Trigger(pChannel, itNoteOnEvent, pChannel->Pitch, pRegion, VoiceType, iKeyGroup) < 0) { dmsg(4,("Voice not triggered\n")); - pKey->pActiveVoices->free(itNewVoice); + GetVoicePool()->free(itNewVoice); } else { // on success --VoiceSpawnsLeft; @@ -1356,8 +2118,26 @@ return -1; } + + /** + * Checks whether scale tuning setting has been changed since last + * time this method was called, if yes, it recalculates the pitch + * for all active voices. + */ + void ProcessScaleTuningChange() { + const bool changed = ScaleTuningChanged.readAndReset(); + if (!changed) return; + + for (int i = 0; i < engineChannels.size(); i++) { + EngineChannelBase* channel = + static_cast*>(engineChannels[i]); + channel->OnScaleTuningChanged(); + } + } private: + Pool< Note >* pNotePool; + Pool noteIDPool; Pool* pVoicePool; ///< Contains all voices that can be activated. Pool SuspendedRegions; Mutex SuspendedRegionsMutex;