si la cosa hubiera sido algo como:

public static string Sumar(string msg, Q1 q)
{ Sumar(msg,q);}

la recursividad hubiera resultado evidente, por contra al usar sobrecarga de operadores resulta que un código realmente simple se convierte en un "quiz", conclusión, la sobrecarga de operadores sirve para generar un código más confuso, menos mantenible y más propenso a errores, mejor no usarla y escribir un método normal y corriente.

Bueno creo que hay que mencionar también que no, el catch no hace captura ninguna puesto que Q1 no hereda de StackOverflowException.

Y una custión que sería bueno resolver es ¿Cuál sería la manera de evitar este tipo de situaciones?

A mi juicio la forma más recomendable es nunca usar el objeto completo para conseguir el resultado de una operación. En el quiz si tuviéramos algo como "Q1 a = new Q1();" y en el la asignación a valor "string valor = "Value: " + a;" también se produciría un StackOverflowException. Por eso mismo usamos el objeto Q1 en su totalidad.

Sería algo como lo que esta implementado en el Quiz al que haces referencia. En la sobrecarga del operador "return i + q.Message.Length;" esto evita un StackOverflowException y lo que se genera es un NullReferenceException. El motivo es que se intenta llegar a una propiedad de un campo de un objeto que no ha sido inicializado "q.Message.Length". Y la forma de evitarlo vendría ser comprobar siempre que los parametros tienen lo que deben tener. O sea, hacer algo cuando se da el siguiente caso "if ((q == null)||(q.Message == null))"