[fixes #2990] Treat records and enums as places where static is all…

…owed.

src/core/lombok/eclipse/handlers/EclipseHandlerUtil.java

@@ -2770,15 +2770,7 @@ static boolean isTypeAndDoesNotHaveFlags(EclipseNode typeNode, long flags) {
 	 * Returns {@code true} if the provided node supports static methods and types (top level or static class)
 	 */
 	public static boolean isStaticAllowed(EclipseNode typeNode) {
-		boolean staticAllowed = true;
-
-		while (typeNode.getKind() != Kind.COMPILATION_UNIT) {
-			if (!staticAllowed) return false;
-
-			staticAllowed = typeNode.isStatic();
-			typeNode = typeNode.up();
-		}
-		return true;
+		return typeNode.isStatic() || typeNode.up() == null || typeNode.up().getKind() == Kind.COMPILATION_UNIT || isRecord(typeNode);
 	}
 
 	public static AbstractVariableDeclaration[] getRecordComponents(TypeDeclaration typeDeclaration) {

src/core/lombok/javac/handlers/HandleLog.java

@@ -80,7 +80,7 @@ public static void processAnnotation(LoggingFramework framework, AnnotationValue
 				return;
 			}
 
-			if (!typeNode.isEnumType() && useStatic && !isStaticAllowed(typeNode)) {
+			if (useStatic && !isStaticAllowed(typeNode)) {
 				annotationNode.addError(framework.getAnnotationAsString() + " is not supported on non-static nested classes.");
 				return;
 			}

src/core/lombok/javac/handlers/JavacHandlerUtil.java

@@ -2096,15 +2096,7 @@ public static boolean isClassAndDoesNotHaveFlags(JavacNode typeNode, long flags)
 	 * Returns {@code true} if the provided node supports static methods and types (top level or static class)
 	 */
 	public static boolean isStaticAllowed(JavacNode typeNode) {
-		boolean staticAllowed = true;
-
-		while (typeNode.getKind() != Kind.COMPILATION_UNIT) {
-			if (!staticAllowed) return false;
-
-			staticAllowed = typeNode.isStatic();
-			typeNode = typeNode.up();
-		}
-		return true;
+		return typeNode.isStatic() || typeNode.up() == null || typeNode.up().getKind() == Kind.COMPILATION_UNIT || isRecord(typeNode);
 	}
 
 	public static JavacNode upToTypeNode(JavacNode node) {

test/transform/resource/after-delombok/LoggerFloggerRecord.java

@@ -0,0 +1,6 @@
+class LoggerFloggerRecord {
+	record Inner(String x) {
+		@java.lang.SuppressWarnings("all")
+		private static final com.google.common.flogger.FluentLogger log = com.google.common.flogger.FluentLogger.forEnclosingClass();
+	}
+}

test/transform/resource/after-ecj/LoggerFloggerRecord.java

@@ -0,0 +1,16 @@
+import lombok.extern.flogger.Flogger;
+class LoggerFloggerRecord {
+  static @Flogger record Inner(String x) {
+    private static final com.google.common.flogger.FluentLogger log = com.google.common.flogger.FluentLogger.forEnclosingClass();
+/* Implicit */    private final String x;
+    <clinit>() {
+    }
+    public Inner(String x) {
+      super();
+      .x = x;
+    }
+  }
+  LoggerFloggerRecord() {
+    super();
+  }
+}

test/transform/resource/before/LoggerFlogger.java

@@ -25,4 +25,4 @@ class LoggerFloggerWithInnerEnum {
 	enum Inner {
 		CONSTANT;
 	}
-}
+}

test/transform/resource/before/LoggerFloggerRecord.java

@@ -0,0 +1,8 @@
+// version 14:
+
+import lombok.extern.flogger.Flogger;
+
+class LoggerFloggerRecord {
+	@Flogger
+	record Inner(String x) {}
+}