somewhat illogical and temporary workaround to the problems with consolidate & bounceable. it turns out that region bouncing never replaces the in-place region, so bounceable() doesn't matter; range bouncing only needs to check bounceable() if we're going to replace material in place and the op is with-processing. reword the error dialog too

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

1 files changed, 4 insertions, 3 deletions

diff --git a/gtk2_ardour/editor_ops.cc b/gtk2_ardour/editor_ops.cc
index 9e46c04eed..b986976f9e 100644
--- a/gtk2_ardour/editor_ops.cc
+++ b/gtk2_ardour/editor_ops.cc
@@ -3319,10 +3319,11 @@ Editor::bounce_range_selection (bool replace, bool enable_processing)
 
 	for (TrackViewList::iterator i = views.begin(); i != views.end(); ++i) {
 		RouteTimeAxisView* rtv = dynamic_cast<RouteTimeAxisView*> (*i);
-		if (rtv && rtv->track() && !rtv->track()->bounceable()) {
+		if (rtv && rtv->track() && replace && enable_processing && !rtv->track()->bounceable()) {
 			MessageDialog d (
-				_("One or more selected tracks cannot be bounced because it has more outputs than inputs.  "
-				  "You can fix this by increasing the number of inputs on that track.")
+				_("You can't perform this operation because the processing of the signal "
+				  "will cause one or more of the tracks will end up with a region with more channels than this track has inputs.\n\n"
+				  "You can do this without processing, which is a different operation.")
 				);
 			d.set_title (_("Cannot bounce"));
 			d.run ();