Fix dumpStackAndThrow; mark with an always-fail contract

Author: Ampflower

src/main/java/net/modfest/fireblanket/diagnostics/ForbiddenStackWalker.java

@@ -2,6 +2,7 @@
 
 import com.mojang.logging.LogUtils;
 import org.jetbrains.annotations.ApiStatus;
+import org.jetbrains.annotations.Contract;
 import org.jetbrains.annotations.NotNull;
 import org.jetbrains.annotations.Nullable;
 import org.slf4j.Logger;
@@ -345,17 +346,17 @@ public static void expectJailbreak() throws IllegalStateException {
 	 * @see #dumpStack(Object...)
 	 */
 	@SuppressWarnings("unused") // API
+	@Contract("_, _ -> fail")
 	public static <T extends Throwable> void dumpStackAndThrow(
 		final @Nullable Supplier<@Nullable T> throwable,
 		final @Nullable Object @Nullable ... context
 	) throws T {
 		if (isAlreadyDumping()) {
 			logger.warn("Terminating recursive call.");
-			return;
+		} else {
+			dump(context);
 		}
 
-		dump(context);
-
 		// If we didn't get an exception provider for some reason,
 		// fall back to throwing an assertion error instead.
 		if (throwable == null) {