Simplify MultitargetException error dumping code

Author: vsilaev

src/main/java/net/tascalate/concurrent/MultitargetException.java

@@ -74,6 +74,7 @@ public static MultitargetException of(final Throwable exception) {
 
     @Override
     public String toString() {
+        // Exclude getMessage / getLocalizedMessage to avoid infinite recursion
         return getClass().getName();
     }
 
@@ -86,15 +87,11 @@ public String getMessage() {
 
     void printDetails(PrintWriter w, Set<Throwable> visited) {
         visited.add(this);
-        w.println(toStringSafe());
+        w.println(this);
         printExceptions(w, (ex, padding) -> {
             PrintWriter pw = new PrintWriter(new PaddedWriter(w, padding), true); 
             if (visited.contains(ex)) {
-                String message = ex instanceof MultitargetException ? 
-                    ((MultitargetException)ex).toStringSafe() 
-                    : 
-                    ex.toString();
-                pw.println("\t[CIRCULAR REFERENCE:" + message + "]");
+                pw.println("\t[CIRCULAR REFERENCE:" + ex + "]");
             } else {
                 if (ex instanceof MultitargetException) {
                     ((MultitargetException)ex).printDetails(pw, visited);
@@ -121,18 +118,14 @@ void printExceptions(PrintStream s, Set<Throwable> visited) {
 
         //super.printStackTrace(s);
         // Print our stack trace
-        s.println(toStringSafe());
+        s.println(this);
         for (StackTraceElement trace : getStackTrace())
             s.println("\tat " + trace);
 
         printExceptions(s, (ex, padding) -> {
             PrintStream ps = new PrintStream(new PaddedOutputStream(s, padding));
             if (visited.contains(ex)) {
-                String message = ex instanceof MultitargetException ? 
-                    ((MultitargetException)ex).toStringSafe() 
-                    : 
-                    ex.toString();
-                ps.println("\t[CIRCULAR REFERENCE:" + message + "]");
+                ps.println("\t[CIRCULAR REFERENCE:" + ex + "]");
             } else {
                 if (ex instanceof MultitargetException) {
                     ((MultitargetException)ex).printExceptions(ps, visited);
@@ -154,18 +147,14 @@ void printExceptions(PrintWriter w, Set<Throwable> visited) {
 
         //super.printStackTrace(s);
         // Print our stack trace
-        w.println(toStringSafe());
+        w.println(this);
         for (StackTraceElement trace : getStackTrace())
             w.println("\tat " + trace);
 
         printExceptions(w, (ex, padding) -> {
             PrintWriter pw = new PrintWriter(new PaddedWriter(w, padding), true); 
             if (visited.contains(ex)) {
-                String message = ex instanceof MultitargetException ? 
-                    ((MultitargetException)ex).toStringSafe() 
-                    : 
-                    ex.toString();
-                pw.println("\t[CIRCULAR REFERENCE:" + message + "]");
+                pw.println("\t[CIRCULAR REFERENCE:" + ex + "]");
             } else {
                 if (ex instanceof MultitargetException) {
                     ((MultitargetException)ex).printExceptions(pw, visited);
@@ -207,10 +196,6 @@ private static <O extends Appendable> void printException(String idx, Throwable
         }
     }
 
-    String toStringSafe() {
-        return getClass().getName();
-    }
-    
     private static <T> Set<T> newDejavueSet() {
         return Collections.newSetFromMap(new IdentityHashMap<T, Boolean>());        
     }