Slightly hacky-fix for misbehaviour when drag-and-dropping processors onto the blank entry (#4668).

git-svn-id: svn://localhost/ardour2/branches/3.0@11385 d708f5d6-7413-0410-9779-e7cbd77b26cf

1 files changed, 15 insertions, 0 deletions

diff --git a/gtk2_ardour/processor_box.cc b/gtk2_ardour/processor_box.cc
index d6697f9de7..9ee94c1793 100644
--- a/gtk2_ardour/processor_box.cc
+++ b/gtk2_ardour/processor_box.cc
@@ -769,6 +769,21 @@ ProcessorBox::object_drop(DnDVBox<ProcessorEntry>* source, ProcessorEntry* posit
 	boost::shared_ptr<Processor> p;
 	if (position) {
 		p = position->processor ();
+		if (!p) {
+			/* dropped on the blank entry (which will be before the
+			   fader), so use the first non-blank child as our
+			   `dropped on' processor */
+			list<ProcessorEntry*> c = processor_display.children ();
+			list<ProcessorEntry*>::iterator i = c.begin ();
+			while (dynamic_cast<BlankProcessorEntry*> (*i)) {
+				assert (i != c.end ());
+				++i;
+			}
+
+			assert (i != c.end ());
+			p = (*i)->processor ();
+			assert (p);
+		}
 	}
 
 	list<ProcessorEntry*> children = source->selection ();