[jOOQ/jOOQ#13581] Remove more unnecessary unary op parentheses

- For more functions with some default emulations
This commit is contained in:
Lukas Eder 2022-05-20 17:39:49 +02:00
parent 2da6648520
commit 14538ec894
80 changed files with 1482 additions and 0 deletions

View File

@ -97,6 +97,7 @@ implements
default:
return true;
}

View File

@ -116,6 +116,7 @@ implements
case MYSQL:
case YUGABYTEDB:
return false;
default:
return true;
}

View File

@ -125,6 +125,7 @@ implements
case SQLITE:
case YUGABYTEDB:
return false;
default:
return true;
}

View File

@ -97,6 +97,7 @@ implements
default:
return true;
}

View File

@ -116,6 +116,7 @@ implements
case MYSQL:
case YUGABYTEDB:
return false;
default:
return true;
}

View File

@ -115,6 +115,7 @@ implements
case MYSQL:
case YUGABYTEDB:
return false;
default:
return true;
}

View File

@ -92,6 +92,27 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case H2:
case HSQLDB:
return true;
case FIREBIRD:
return true;
default:
return false;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -89,6 +89,40 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case DERBY:
case SQLITE:
return false;
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -89,6 +89,29 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case HSQLDB:
return false;
case H2:
return true;
case FIREBIRD:
return true;
default:
return false;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -92,6 +92,31 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case H2:
case HSQLDB:
return true;
case FIREBIRD:
return true;
default:
return false;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -92,6 +92,44 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case CUBRID:
case FIREBIRD:
case HSQLDB:
case MARIADB:
case MYSQL:
case POSTGRES:
case SQLITE:
case YUGABYTEDB:
return false;
case H2:
return false;
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -92,6 +92,30 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case H2:
case HSQLDB:
return true;
case FIREBIRD:
return true;
case SQLITE:
return false;
default:
return false;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -89,6 +89,28 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case SQLITE:
return false;
case H2:
return true;
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -90,6 +90,34 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case CUBRID:
case HSQLDB:
case MARIADB:
case MYSQL:
case POSTGRES:
case YUGABYTEDB:
return false;
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -90,6 +90,23 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case SQLITE:
return false;
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -90,6 +90,47 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case CUBRID:
case DERBY:
case FIREBIRD:
case H2:
case HSQLDB:
case IGNITE:
case MARIADB:
case MYSQL:
case POSTGRES:
case SQLITE:
case YUGABYTEDB:
return false;
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -90,6 +90,29 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case FIREBIRD:
return false;
case SQLITE:
return false;
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -84,6 +84,49 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case CUBRID:
case DERBY:
case FIREBIRD:
case H2:
case HSQLDB:
case IGNITE:
case MARIADB:
case MYSQL:
case POSTGRES:
case YUGABYTEDB:
return false;
case SQLITE:
return false;
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -90,6 +90,19 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -89,6 +89,22 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case SQLITE:
return false;
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -92,6 +92,19 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case DERBY:
case SQLITE:
return false;
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -90,6 +90,29 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case SQLITE:
return true;
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -90,6 +90,32 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case YUGABYTEDB:
return false;
case POSTGRES:
case SQLITE:
return false;
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -92,6 +92,27 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case SQLITE:
return false;
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -92,6 +92,19 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -92,6 +92,50 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case CUBRID:
case DERBY:
case FIREBIRD:
case IGNITE:
case POSTGRES:
case YUGABYTEDB:
return true;
case MARIADB:
case MYSQL:
case SQLITE:
return true;
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -84,6 +84,30 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case SQLITE:
return false;
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -93,6 +93,22 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case DERBY:
return false;
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -90,6 +90,25 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case FIREBIRD:
return false;
case SQLITE:
return false;
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -84,6 +84,37 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case DERBY:
case POSTGRES:
case SQLITE:
case YUGABYTEDB:
return true;
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -92,6 +92,33 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case FIREBIRD:
return false;
case SQLITE:
return false;
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -92,6 +92,21 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case DERBY:
return false;
case SQLITE:
return false;
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -92,6 +92,39 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case FIREBIRD:
return true;
case H2:
return true;
case HSQLDB:
return false;
default:
return false;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -92,6 +92,39 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case FIREBIRD:
return true;
case H2:
return true;
case HSQLDB:
return false;
default:
return false;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -89,6 +89,22 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case SQLITE:
return false;
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -90,6 +90,34 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case CUBRID:
case HSQLDB:
case MARIADB:
case MYSQL:
case POSTGRES:
case YUGABYTEDB:
return false;
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -89,6 +89,34 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case FIREBIRD:
case SQLITE:
return false;
case DERBY:
case HSQLDB:
case POSTGRES:
case YUGABYTEDB:
return false;
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -95,6 +95,32 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case MARIADB:
case MYSQL:
return false;
case HSQLDB:
return false;
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -90,6 +90,22 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -89,6 +89,46 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case SQLITE:
return false;
case CUBRID:
case DERBY:
case FIREBIRD:
case H2:
case HSQLDB:
case IGNITE:
case MARIADB:
case MYSQL:
case POSTGRES:
case YUGABYTEDB:
return false;
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -156,6 +156,20 @@ package org.jooq.impl;

View File

@ -156,6 +156,20 @@ package org.jooq.impl;

View File

@ -156,6 +156,20 @@ package org.jooq.impl;

View File

@ -156,6 +156,20 @@ package org.jooq.impl;

View File

@ -170,6 +170,20 @@ package org.jooq.impl;

View File

@ -169,6 +169,17 @@ package org.jooq.impl;

View File

@ -174,6 +174,20 @@ package org.jooq.impl;

View File

@ -169,6 +169,17 @@ package org.jooq.impl;

View File

@ -174,6 +174,20 @@ package org.jooq.impl;

View File

@ -155,6 +155,17 @@ package org.jooq.impl;

View File

@ -170,6 +170,20 @@ package org.jooq.impl;

View File

@ -156,6 +156,20 @@ package org.jooq.impl;

View File

@ -186,6 +186,20 @@ package org.jooq.impl;

View File

@ -173,6 +173,17 @@ package org.jooq.impl;

View File

@ -167,6 +167,20 @@ package org.jooq.impl;

View File

@ -186,6 +186,20 @@ package org.jooq.impl;

View File

@ -174,6 +174,20 @@ package org.jooq.impl;

View File

@ -174,6 +174,20 @@ package org.jooq.impl;

View File

@ -170,6 +170,20 @@ package org.jooq.impl;

View File

@ -151,6 +151,17 @@ package org.jooq.impl;

View File

@ -151,6 +151,17 @@ package org.jooq.impl;

View File

@ -156,6 +156,20 @@ package org.jooq.impl;

View File

@ -156,6 +156,20 @@ package org.jooq.impl;

View File

@ -157,6 +157,23 @@ package org.jooq.impl;

View File

@ -155,6 +155,17 @@ package org.jooq.impl;

View File

@ -170,6 +170,20 @@ package org.jooq.impl;

View File

@ -173,6 +173,17 @@ package org.jooq.impl;

View File

@ -156,6 +156,20 @@ package org.jooq.impl;

View File

@ -155,6 +155,17 @@ package org.jooq.impl;

View File

@ -170,6 +170,20 @@ package org.jooq.impl;

View File

@ -178,6 +178,23 @@ package org.jooq.impl;

View File

@ -170,6 +170,20 @@ package org.jooq.impl;

View File

@ -156,6 +156,20 @@ package org.jooq.impl;

View File

@ -156,6 +156,20 @@ package org.jooq.impl;

View File

@ -156,6 +156,20 @@ package org.jooq.impl;

View File

@ -95,6 +95,21 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -90,6 +90,34 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case CUBRID:
case HSQLDB:
case MARIADB:
case MYSQL:
case POSTGRES:
case YUGABYTEDB:
return false;
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -89,6 +89,30 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case MARIADB:
case MYSQL:
return true;
case H2:
return false;
case SQLITE:
return false;
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -95,6 +95,22 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {

View File

@ -84,6 +84,70 @@ implements
// XXX: QueryPart API
// -------------------------------------------------------------------------
@Override
final boolean parenthesised(Context<?> ctx) {
switch (ctx.family()) {
case POSTGRES:
return true;
case FIREBIRD:
return false;
case H2:
return true;
case HSQLDB:
case IGNITE:
case MARIADB:
case MYSQL:
return true;
case SQLITE:
return false;
default:
return true;
}
}
@Override
public final void accept(Context<?> ctx) {
switch (ctx.family()) {