[jOOQ/jOOQ#11083] DataType should cache null_() and notNull() variants

2024-02-29 08:53:18 +01:00 · 2024-02-29 08:53:18 +01:00 · 1740abe739
commit 1740abe739
parent 181d838797
1 changed files with 18 additions and 15 deletions
--- a/jOOQ/src/main/java/org/jooq/impl/AbstractDataTypeX.java
+++ b/jOOQ/src/main/java/org/jooq/impl/AbstractDataTypeX.java
@ -85,21 +85,24 @@ abstract class AbstractDataTypeX<T> extends AbstractDataType<T> {

    @Override
    public /* non-final */ DataType<T> nullability(Nullability n) {
-        return construct(
-            precision0(),
-            scale0(),
-            length0(),
-            n,
-            hidden(),
-            readonly(),
-            generatedAlwaysAsGenerator(),
-            generationOption(),
-            generationLocation(),
-            collation(),
-            characterSet(),
-            !n.nullable() && identity(),
-            defaultValue()
-        );
+        if (n == nullability())
+            return this;
+        else
+            return construct(
+                precision0(),
+                scale0(),
+                length0(),
+                n,
+                hidden(),
+                readonly(),
+                generatedAlwaysAsGenerator(),
+                generationOption(),
+                generationLocation(),
+                collation(),
+                characterSet(),
+                !n.nullable() && identity(),
+                defaultValue()
+            );
    }

    @Override