From 11d604dea3a4be4bffa3a2c5a9b80ae03f39512c Mon Sep 17 00:00:00 2001
From: Carl Hetherington <carl@carlh.net>
Date: Thu, 3 Nov 2011 02:20:51 +0000
Subject: [PATCH] Remove edit option from internal send context menus; it's
 pretty confusing, and there is a fader just underneath the send processor's
 label to edit its level.

git-svn-id: svn://localhost/ardour2/branches/3.0@10409 d708f5d6-7413-0410-9779-e7cbd77b26cf
---
 gtk2_ardour/processor_box.cc | 16 +---------------
 1 file changed, 1 insertion(+), 15 deletions(-)

diff --git a/gtk2_ardour/processor_box.cc b/gtk2_ardour/processor_box.cc
index 883ce4b196..bb09aba272 100644
--- a/gtk2_ardour/processor_box.cc
+++ b/gtk2_ardour/processor_box.cc
@@ -1780,7 +1780,7 @@ ProcessorBox::processor_can_be_edited (boost::shared_ptr<Processor> processor)
 	}
 
 	if (
-		boost::dynamic_pointer_cast<Send> (processor) ||
+		(boost::dynamic_pointer_cast<Send> (processor) && !boost::dynamic_pointer_cast<InternalSend> (processor))||
 		boost::dynamic_pointer_cast<Return> (processor) ||
 		boost::dynamic_pointer_cast<PluginInsert> (processor) ||
 		boost::dynamic_pointer_cast<PortInsert> (processor)
@@ -1826,20 +1826,6 @@ ProcessorBox::toggle_edit_processor (boost::shared_ptr<Processor> processor)
 			_parent_strip->revert_to_default_display ();
 		}
 
-	} else if ((internal_send = boost::dynamic_pointer_cast<InternalSend> (processor)) != 0) {
-
-		if (!_session->engine().connected()) {
-			return;
-		}
-
-		if (_parent_strip) {
-			if (boost::dynamic_pointer_cast<Send> (_parent_strip->current_delivery()) == internal_send) {
-				_parent_strip->revert_to_default_display ();
-			} else {
-				_parent_strip->show_send (internal_send);
-			}
-		}
-
 	} else if ((send = boost::dynamic_pointer_cast<Send> (processor)) != 0) {
 
 		if (!_session->engine().connected()) {