added autoexpand to steprec.cpp

reformatted StepRec::record to Allman style ("my" style)
fixed a bug in steprec.cpp: parts with tick() being !=0 were 
treated wrong

1 files changed, 136 insertions, 112 deletions

diff --git a/muse2/muse/steprec.cpp b/muse2/muse/steprec.cpp
index 29cb9540..c1fc23b1 100644
--- a/muse2/muse/steprec.cpp
+++ b/muse2/muse/steprec.cpp
@@ -9,6 +9,7 @@
 #include "part.h"
 #include "event.h"
 #include "globals.h"
+#include "functions.h"
 
 #include "song.h"
 #include "audio.h"
@@ -39,121 +40,144 @@ void StepRec::timeout()
 
 void StepRec::record(Part* part, int pitch, int len, int step, int velo, bool ctrl, bool shift)
 {
-	       unsigned tick = song->cpos();
-	       
-         if (pitch!=rcSteprecNote) {
-					  chord_timer->stop();
-					  
-            
-            //
-            // extend len of last note?
-            //
-            EventList* events = part->events();
-            if (ctrl) {
-                  for (iEvent i = events->begin(); i != events->end(); ++i) {
-                        Event ev = i->second;
-                        if (!ev.isNote())
-                              continue;
-                        if (ev.pitch() == pitch && ((ev.tick() + ev.lenTick()) == tick)) {
-                              Event e = ev.clone();
-                              e.setLenTick(ev.lenTick() + len);
-                              // Indicate do undo, and do not do port controller values and clone parts. 
-                              audio->msgChangeEvent(ev, e, part, true, false, false);
-                              
-                              if (!shift) {
-                                    chord_timer_set_to_tick = tick + step;
-                                    chord_timer->start();
-                                    }
-                              return;
-                              }
-                        }
-                  }
+	unsigned tick = song->cpos();
+	unsigned lasttick=0;
+	Undo operations;
+	
+	if (pitch!=rcSteprecNote) 
+	{
+		chord_timer->stop();
+
+		// extend len of last note?
+		EventList* events = part->events();
+		if (ctrl)
+		{
+			for (iEvent i = events->begin(); i != events->end(); ++i)
+			{
+				Event ev = i->second;
+				if (ev.isNote() && ev.pitch() == pitch && ((ev.tick() + ev.lenTick() + part->tick()) == tick))
+				{
+					Event e = ev.clone();
+					e.setLenTick(ev.lenTick() + len);
+					operations.push_back(UndoOp(UndoOp::ModifyEvent, e,ev, part, false, false));
+					
+					if (!shift)
+					{
+						chord_timer_set_to_tick = tick + step;
+						chord_timer->start();
+					}
+					
+					lasttick=tick+len - part->tick();
+					goto steprec_record_foot;
+				}
+			}
+		}
 
-            //
-            // if we already entered the note, delete it
-            //
-            EventRange range = events->equal_range(tick);
-            for (iEvent i = range.first; i != range.second; ++i) {
-                  Event ev = i->second;
-                  if (ev.isNote() && ev.pitch() == pitch) {
-                        // Indicate do undo, and do not do port controller values and clone parts. 
-                        //audio->msgDeleteEvent(ev, part);
-                        audio->msgDeleteEvent(ev, part, true, false, false);
+		if (tick<=part->endTick())
+		{
+			// if we already entered the note, delete it
+			// if we would find a note after part->lenTick(), the above "if"
+			// avoids this. this has to be avoided because then part->hasHiddenNotes() is true
+			// which results in forbidding any action beyond its end
+			EventRange range = events->equal_range(tick - part->tick());
+			for (iEvent i = range.first; i != range.second; ++i)
+			{
+				Event ev = i->second;
+				if (ev.isNote() && ev.pitch() == pitch)
+				{
+					audio->msgDeleteEvent(ev, part, true, false, false);
 
-                        if (!shift) {
-                              chord_timer_set_to_tick = tick + step;
-                              chord_timer->start();
-                              }
-                        
-                        return;
-                        }
-                  }
-                  
-            Event e(Note);
-            e.setTick(tick - part->tick());
-            e.setPitch(pitch);
-            e.setVelo(velo);
-            e.setLenTick(len);
-            // Indicate do undo, and do not do port controller values and clone parts. 
-            //audio->msgAddEvent(e, part);
-            audio->msgAddEvent(e, part, true, false, false);
-            
-            if (! (globalKeyState & Qt::ShiftModifier)) {
-                  chord_timer_set_to_tick = tick + step;
-                  chord_timer->start();
-                  }
-            }
-         else { // equals if (pitch==rcSteprecNote)
-            bool held_notes=false;
-            if (note_held_down!=NULL)
-            {
-							for (int i=0;i<128;i++)
-								 if (note_held_down[i]) { held_notes=true; break; }
-						}
-						else
-							held_notes=false;
-								 
-            
-            if (held_notes)
-            {
-                chord_timer->stop();
-                
-                // extend len of last note(s)
-                using std::set;
-                
-                set<Event*> extend_set;
-                EventList* events = part->events();
-                for (iEvent i = events->begin(); i != events->end(); ++i) {
-                      Event& ev = i->second;
-                      if (!ev.isNote())
-                            continue;
+					if (!shift)
+					{
+						chord_timer_set_to_tick = tick + step;
+						chord_timer->start();
+					}
+					
+					return;
+				}
+			}
+		}
+		
+				
+		Event e(Note);
+		e.setTick(tick - part->tick());
+		e.setPitch(pitch);
+		e.setVelo(velo);
+		e.setLenTick(len);
+		operations.push_back(UndoOp(UndoOp::AddEvent, e, part, false, false));
+		lasttick=e.endTick();
+
+		if (! (globalKeyState & Qt::ShiftModifier))
+		{
+			chord_timer_set_to_tick = tick + step;
+			chord_timer->start();
+		}
+		
+		goto steprec_record_foot; // this is actually unneccessary, but for clarity
+	}
+	else  // equals if (pitch==rcSteprecNote)
+	{
+		bool held_notes=false;
+		if (note_held_down!=NULL)
+		{
+			for (int i=0;i<128;i++)
+			if (note_held_down[i]) { held_notes=true; break; }
+		}
+		else
+			held_notes=false;
+			 
 
-                      if (note_held_down[ev.pitch()] && ((ev.tick() + ev.lenTick()) == tick))
-                            extend_set.insert(&ev);
-                      }
-                for (set<Event*>::iterator it=extend_set.begin(); it!=extend_set.end(); it++)
-                {
-                    Event& ev=**it;
-                    Event e = ev.clone();
-                    e.setLenTick(ev.lenTick() + len);
-                    // Indicate do undo, and do not do port controller values and clone parts. 
-                    audio->msgChangeEvent(ev, e, part, true, false, false);
-                }
+		if (held_notes)
+		{
+			chord_timer->stop();
+			
+			// extend len of last note(s)
+			using std::set;
+			
+			set<Event*> extend_set;
+			EventList* events = part->events();
+			for (iEvent i = events->begin(); i != events->end(); ++i)
+			{
+				Event& ev = i->second;
+				if (ev.isNote() && note_held_down[ev.pitch()] && ((ev.tick() + ev.lenTick() + part->tick()) == tick))
+					extend_set.insert(&ev);
+			}
+			
+			for (set<Event*>::iterator it=extend_set.begin(); it!=extend_set.end(); it++)
+			{
+				Event& ev=**it;
+				Event e = ev.clone();
+				e.setLenTick(ev.lenTick() + len);
+				operations.push_back(UndoOp(UndoOp::ModifyEvent,e, ev, part, false, false));
+			}
 
-                if (!shift) {
-                      chord_timer_set_to_tick = tick + step;
-                      chord_timer->start();
-                      }
-                return;
-               
-            }
-            else // equals if (!held_notes)
-            {
-              chord_timer->stop();
+			if (!shift)
+			{
+				chord_timer_set_to_tick = tick + step;
+				chord_timer->start();
+			}
+			
+			lasttick=tick+len - part->tick();
+			goto steprec_record_foot; // this is actually unneccessary, but for clarity
+		}
+		else // equals if (!held_notes)
+		{
+			chord_timer->stop();
 
-              //simply proceed, inserting a rest
-              Pos p(song->cpos() + step, true);
-              song->setPos(0, p, true, false, true);
-            }
-            }
+			// simply proceed, inserting a rest
+			Pos p(song->cpos() + step, true);
+			song->setPos(0, p, true, false, true);
+			
+			return;
+		}
+	}
+	
+	steprec_record_foot:
+	if (!((lasttick > part->lenTick()) && part->hasHiddenNotes())) // allowed?
+	{
+		if (lasttick > part->lenTick()) // we have to expand the part?
+			schedule_resize_all_same_len_clone_parts(part, lasttick, operations);
+		
+		song->applyOperationGroup(operations);
+	}
 }