router_route_intern: stop processing if a group caused a stop

In issue #246, the optimiser created a match group, followed by a
matchall rule.  The stop from the match group did not cause an abort of
processing any further rules /after/ the group, which is wrong
behaviour.  With this commit that behaviour is fixed.

ChangeLog.md

@@ -11,6 +11,12 @@
 * **relay** A new `-O` flag allows to set the minimum rules treshold
             before trying to optimise the rules.
 
+### Bugfixes
+
+* [Issue #246](https://github.com/grobian/carbon-c-relay/issues/246)
+  using `stop` in a `match` tule does not stop metrics from being
+  processed afterwards
+
 
 # 2.5 (2017-01-09)
 

router.c

@@ -3184,11 +3184,11 @@ router_route_intern(
 					}	break;
 				}
 			}
-
-			/* stop processing further rules if requested */
-			if (stop)
-				break;
 		}
+
+		/* stop processing further rules if requested */
+		if (stop)
+			break;
 	}
 	if (!wassent)
 		*blackholed = 1;