IJ-CR-153375 [java-decompiler] IDEA-289211 set limits for some cases in decompiler

- change messages
- add test cases

GitOrigin-RevId: 8347f136818567abccf56a4c110b77456870fd4e

Author: Mikhail Pyltsin

src/org/jetbrains/java/decompiler/main/collectors/LimitContainer.java

@@ -8,45 +8,41 @@
 import java.util.Map;
 import java.util.concurrent.atomic.AtomicLong;
 
-import static org.jetbrains.java.decompiler.main.extern.IFernflowerPreferences.*;
+import static org.jetbrains.java.decompiler.main.extern.IFernflowerPreferences.MAX_DIRECT_NODES_COUNT;
+import static org.jetbrains.java.decompiler.main.extern.IFernflowerPreferences.MAX_DIRECT_VARIABLE_NODE_COUNT;
 
 public class LimitContainer {
 
   private final int maxDirectNodeCount;//-1 - don't check
   @NotNull
   private final AtomicLong directNodeCount = new AtomicLong();
-  @NotNull
-  private final String maxDirectNodeCountMessage;
 
   private final int ssaConstructorSparseExRecordCount;//-1 - don't check
-  @NotNull
-  private final String ssaConstructorSparseExRecordCountMessage;
 
   public LimitContainer(@NotNull Map<String, Object> properties) {
     maxDirectNodeCount = (int)properties.getOrDefault(MAX_DIRECT_NODES_COUNT, -1);
-    maxDirectNodeCountMessage = (String)properties.getOrDefault(MAX_DIRECT_NODES_COUNT_MESSAGE, "Limits are exceeded");
     ssaConstructorSparseExRecordCount = (int)properties.getOrDefault(MAX_DIRECT_VARIABLE_NODE_COUNT, -1);
-    ssaConstructorSparseExRecordCountMessage = (String)properties.getOrDefault(MAX_DIRECT_VARIABLE_NODES_COUNT_MESSAGE, "Limits are exceeded");
   }
 
   public void incrementAndCheckDirectNodeCount(@NotNull ControlFlowGraph graph) {
     long newValue = directNodeCount.addAndGet(graph.getBlocks().size());
     if (maxDirectNodeCount != -1 && newValue >= maxDirectNodeCount) {
-      throw new LimitExceededDecompilerException(maxDirectNodeCountMessage);
+      throw new LimitExceededDecompilerException(maxDirectNodeCount, newValue, "direct nodes");
     }
   }
 
   public void checkSFormsFastMapDirect(@NotNull Map<String, SFormsFastMapDirect> inVarVersions,
                                        @NotNull Map<String, SFormsFastMapDirect> outVarVersions) {
+    int newValue = inVarVersions.size() + outVarVersions.size();
     if (ssaConstructorSparseExRecordCount != -1 &&
-        inVarVersions.size() + outVarVersions.size() > ssaConstructorSparseExRecordCount) {
-      throw new LimitExceededDecompilerException(ssaConstructorSparseExRecordCountMessage);
+        newValue > ssaConstructorSparseExRecordCount) {
+      throw new LimitExceededDecompilerException(ssaConstructorSparseExRecordCount, newValue, "variable nodes");
     }
   }
 
   public static class LimitExceededDecompilerException extends RuntimeException {
-    public LimitExceededDecompilerException(String message) {
-      super(message);
+    public LimitExceededDecompilerException(long limit, long actualValue, String type) {
+      super("Limits for %s are exceeded. Current value: %s, limit: %s".formatted(type, actualValue, limit));
     }
   }
 }

src/org/jetbrains/java/decompiler/main/extern/IFernflowerPreferences.java

@@ -64,9 +64,7 @@ public interface IFernflowerPreferences {
   String SKIP_EXTRA_FILES = "sef";
 
   String MAX_DIRECT_NODES_COUNT = "mdnc";
-  String MAX_DIRECT_NODES_COUNT_MESSAGE = "mdncm";
   String MAX_DIRECT_VARIABLE_NODE_COUNT = "mdvnc";
-  String MAX_DIRECT_VARIABLE_NODES_COUNT_MESSAGE = "mdvncm";
 
   Map<String, String> DEFAULTS = getDefaults();