diff --git a/jOOQ/src/main/java/org/jooq/Field.java b/jOOQ/src/main/java/org/jooq/Field.java index f025affc28..cada5124a6 100644 --- a/jOOQ/src/main/java/org/jooq/Field.java +++ b/jOOQ/src/main/java/org/jooq/Field.java @@ -709,6 +709,13 @@ extends @Support({ CUBRID, FIREBIRD, H2, HSQLDB, MARIADB, MYSQL, POSTGRES, SQLITE, YUGABYTE }) Field bitNor(Field arg2); + /** + * The BIT_NOT operator. + */ + @NotNull + @Support({ CUBRID, FIREBIRD, H2, HSQLDB, MARIADB, MYSQL, POSTGRES, SQLITE, YUGABYTE }) + Field bitNot(); + /** * The BIT_X_NOR operator. * @@ -1173,15 +1180,6 @@ extends // Bitwise operations // ------------------------------------------------------------------------ - /** - * The bitwise not operator. - * - * @see DSL#bitNot(Field) - */ - @NotNull - @Support({ CUBRID, FIREBIRD, H2, HSQLDB, MARIADB, MYSQL, POSTGRES, SQLITE, YUGABYTE }) - Field bitNot(); - /** * The bitwise and operator. * diff --git a/jOOQ/src/main/java/org/jooq/impl/AbstractField.java b/jOOQ/src/main/java/org/jooq/impl/AbstractField.java index 12fd1afd50..fd88d62d50 100644 --- a/jOOQ/src/main/java/org/jooq/impl/AbstractField.java +++ b/jOOQ/src/main/java/org/jooq/impl/AbstractField.java @@ -363,6 +363,12 @@ abstract class AbstractField extends AbstractTypedNamed implements Field) arg2); } + @Override + @SuppressWarnings({ "unchecked", "rawtypes" }) + public final Field bitNot() { + return DSL.bitNot((Field) this); + } + @Override @SuppressWarnings({ "unchecked", "rawtypes" }) public final Field bitXNor(T arg2) { @@ -560,12 +566,6 @@ abstract class AbstractField extends AbstractTypedNamed implements Field bitNot() { - return DSL.bitNot((Field) this); - } - @SuppressWarnings({ "unchecked", "rawtypes" }) @Override public final Field bitAnd(T value) { diff --git a/jOOQ/src/main/java/org/jooq/impl/BitNot.java b/jOOQ/src/main/java/org/jooq/impl/BitNot.java new file mode 100644 index 0000000000..a52a908cd7 --- /dev/null +++ b/jOOQ/src/main/java/org/jooq/impl/BitNot.java @@ -0,0 +1,144 @@ +/* + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + * + * Other licenses: + * ----------------------------------------------------------------------------- + * Commercial licenses for this work are available. These replace the above + * ASL 2.0 and offer limited warranties, support, maintenance, and commercial + * database integrations. + * + * For more information, please visit: http://www.jooq.org/licenses + * + * + * + * + * + * + * + * + * + * + * + * + * + * + * + * + */ +package org.jooq.impl; + +import static org.jooq.impl.DSL.*; +import static org.jooq.impl.Internal.*; +import static org.jooq.impl.Keywords.*; +import static org.jooq.impl.Names.*; +import static org.jooq.impl.SQLDataType.*; +import static org.jooq.impl.Tools.*; +import static org.jooq.impl.Tools.BooleanDataKey.*; +import static org.jooq.impl.Tools.DataExtendedKey.*; +import static org.jooq.impl.Tools.DataKey.*; +import static org.jooq.SQLDialect.*; + +import org.jooq.*; +import org.jooq.Record; +import org.jooq.conf.*; +import org.jooq.impl.*; +import org.jooq.tools.*; + +import java.util.*; + + +/** + * The BIT NOT statement. + */ +@SuppressWarnings({ "rawtypes", "unchecked", "unused" }) +final class BitNot +extends + AbstractField +{ + + private final Field value; + + BitNot( + Field value + ) { + super( + N_BIT_NOT, + allNotNull((DataType) dataType(INTEGER, value, false), value) + ); + + this.value = nullSafeNotNull(value, INTEGER); + } + + // ------------------------------------------------------------------------- + // XXX: QueryPart API + // ------------------------------------------------------------------------- + + @Override + public final void accept(Context ctx) { + switch (ctx.family()) { + + + + + + + + + case HSQLDB: + ctx.visit(isub(isub(zero(), value), one())); + break; + + + + + + case H2: + ctx.visit(function(N_BITNOT, getDataType(), value)); + break; + + case FIREBIRD: + ctx.visit(function(N_BIN_NOT, getDataType(), value)); + break; + + default: + ctx.sql("~(").visit(value).sql(')'); + break; + } + } + + + + + + + + + + + + + // ------------------------------------------------------------------------- + // The Object API + // ------------------------------------------------------------------------- + + @Override + public boolean equals(Object that) { + if (that instanceof BitNot) { + return + StringUtils.equals(value, ((BitNot) that).value) + ; + } + else + return super.equals(that); + } +} diff --git a/jOOQ/src/main/java/org/jooq/impl/DSL.java b/jOOQ/src/main/java/org/jooq/impl/DSL.java index f96e9f5ef0..1b9ec5cb4d 100644 --- a/jOOQ/src/main/java/org/jooq/impl/DSL.java +++ b/jOOQ/src/main/java/org/jooq/impl/DSL.java @@ -15926,6 +15926,26 @@ public class DSL { return new BitNor(arg1, arg2); } + /** + * The BIT_NOT function. + * + * @param value is wrapped as {@link #val(Object)}. + */ + @NotNull + @Support({ CUBRID, FIREBIRD, H2, HSQLDB, MARIADB, MYSQL, POSTGRES, SQLITE, YUGABYTE }) + public static Field bitNot(T value) { + return new BitNot(Tools.field(value)); + } + + /** + * The BIT_NOT function. + */ + @NotNull + @Support({ CUBRID, FIREBIRD, H2, HSQLDB, MARIADB, MYSQL, POSTGRES, SQLITE, YUGABYTE }) + public static Field bitNot(Field value) { + return new BitNot(value); + } + /** * The BIT_X_NOR function. * @@ -22535,29 +22555,6 @@ public class DSL { // XXX Bitwise operations // ------------------------------------------------------------------------ - /** - * The bitwise not operator. - * - * @see #bitNot(Field) - */ - @NotNull - @Support({ CUBRID, FIREBIRD, H2, HSQLDB, MARIADB, MYSQL, POSTGRES, SQLITE, YUGABYTE }) - public static Field bitNot(T value) { - return bitNot(Tools.field(value)); - } - - /** - * The bitwise not operator. - *

- * Most dialects natively support this using ~[field]. jOOQ - * emulates this operator in some dialects using -[field] - 1 - */ - @NotNull - @Support({ CUBRID, FIREBIRD, H2, HSQLDB, MARIADB, MYSQL, POSTGRES, SQLITE, YUGABYTE }) - public static Field bitNot(Field field) { - return new Neg<>(Tools.nullSafe(field), false, ExpressionOperator.BIT_NOT); - } - /** * The bitwise and operator. * diff --git a/jOOQ/src/main/java/org/jooq/impl/Expression.java b/jOOQ/src/main/java/org/jooq/impl/Expression.java index b576fcff89..778ce354b9 100644 --- a/jOOQ/src/main/java/org/jooq/impl/Expression.java +++ b/jOOQ/src/main/java/org/jooq/impl/Expression.java @@ -252,7 +252,6 @@ final class Expression extends AbstractTransformable { - @Override diff --git a/jOOQ/src/main/java/org/jooq/impl/ExpressionOperator.java b/jOOQ/src/main/java/org/jooq/impl/ExpressionOperator.java index d354b3be4d..2bb3ba0d9b 100644 --- a/jOOQ/src/main/java/org/jooq/impl/ExpressionOperator.java +++ b/jOOQ/src/main/java/org/jooq/impl/ExpressionOperator.java @@ -76,11 +76,6 @@ enum ExpressionOperator { */ MODULO("%"), - /** - * Bitwise not - */ - BIT_NOT("~"), - /** * Bitwise and */ diff --git a/jOOQ/src/main/java/org/jooq/impl/Names.java b/jOOQ/src/main/java/org/jooq/impl/Names.java index 165f5a6e58..6bb971a5b4 100644 --- a/jOOQ/src/main/java/org/jooq/impl/Names.java +++ b/jOOQ/src/main/java/org/jooq/impl/Names.java @@ -96,6 +96,7 @@ final class Names { static final Name N_BIT_LENGTH = unquotedName("bit_length"); static final Name N_BIT_NAND = unquotedName("bit_nand"); static final Name N_BIT_NOR = unquotedName("bit_nor"); + static final Name N_BIT_NOT = unquotedName("bit_not"); static final Name N_BIT_OR = unquotedName("bit_or"); static final Name N_BIT_OR_AGG = unquotedName("bit_or_agg"); static final Name N_BIT_XOR = unquotedName("bit_xor"); diff --git a/jOOQ/src/main/java/org/jooq/impl/Neg.java b/jOOQ/src/main/java/org/jooq/impl/Neg.java index 5080a08731..c1aa93a27c 100644 --- a/jOOQ/src/main/java/org/jooq/impl/Neg.java +++ b/jOOQ/src/main/java/org/jooq/impl/Neg.java @@ -38,29 +38,10 @@ package org.jooq.impl; // ... -import static org.jooq.SQLDialect.FIREBIRD; -import static org.jooq.SQLDialect.H2; -import static org.jooq.SQLDialect.HSQLDB; -// ... -// ... -// ... -// ... -// ... -// ... -import static org.jooq.impl.DSL.function; -import static org.jooq.impl.DSL.one; -import static org.jooq.impl.DSL.zero; -import static org.jooq.impl.ExpressionOperator.BIT_NOT; -import static org.jooq.impl.Internal.isub; -import static org.jooq.impl.Names.N_BIN_NOT; -import static org.jooq.impl.Names.N_BITNOT; - -import java.util.Set; import org.jooq.Context; import org.jooq.Field; import org.jooq.Param; -import org.jooq.SQLDialect; import org.jooq.conf.TransformUnneededArithmeticExpressions; /** @@ -68,9 +49,6 @@ import org.jooq.conf.TransformUnneededArithmeticExpressions; */ final class Neg extends AbstractTransformable { - private static final Set EMULATE_BIT_NOT = SQLDialect.supportedBy(HSQLDB); - private static final Set SUPPORT_BIT_NOT = SQLDialect.supportedBy(H2); - private final Field field; private final boolean internal; private final ExpressionOperator operator; @@ -110,19 +88,11 @@ final class Neg extends AbstractTransformable { @Override public final void accept0(Context ctx) { - SQLDialect family = ctx.family(); - - if (operator == BIT_NOT && EMULATE_BIT_NOT.contains(ctx.dialect())) - ctx.visit(isub(isub(zero(), field), one())); - else if (operator == BIT_NOT && SUPPORT_BIT_NOT.contains(ctx.dialect())) - ctx.visit(function(N_BITNOT, getDataType(), field)); - else if (operator == BIT_NOT && family == FIREBIRD) - ctx.visit(function(N_BIN_NOT, getDataType(), field)); else ctx.sql(operator.toSQL()) .sql('(')