Re: [PATCH] sql: add missing postgresql types

[Robert Pluim]

>>>>> On Sun, 16 Apr 2023 15:38:43 +0200, Nicolas Martyanoff <nicolas@n16f.net> 
>>>>> said:

    Nicolas> * lisp/progmodes/sql.el: Add multiple missing PostgreSQL types: 
bigserial,
    Nicolas> smallserial, macaddr8, jsonb, jsonpath, built-in range types, 
object
    Nicolas> identifier types (oid & co).
    Nicolas> ---
    Nicolas>  lisp/progmodes/sql.el | 19 +++++++++++--------
    Nicolas>  1 file changed, 11 insertions(+), 8 deletions(-)

    Nicolas> diff --git a/lisp/progmodes/sql.el b/lisp/progmodes/sql.el
    Nicolas> index d8bd6bc2685..75d0cbbb367 100644
    Nicolas> --- a/lisp/progmodes/sql.el
    Nicolas> +++ b/lisp/progmodes/sql.el
    Nicolas> @@ -2192,14 +2192,17 @@ sql-mode-postgres-font-lock-keywords
 
    Nicolas>       ;; Postgres Data Types
    Nicolas>       (sql-font-lock-keywords-builder 'font-lock-type-face nil
    Nicolas> -"bigint" "bigserial" "bit" "bool" "boolean" "box" "bytea" "char"
    Nicolas> -"character" "cidr" "circle" "date" "decimal" "double" "float4"
    Nicolas> -"float8" "inet" "int" "int2" "int4" "int8" "integer" "interval" 
"line"
    Nicolas> -"lseg" "macaddr" "money" "name" "numeric" "path" "point" "polygon"
    Nicolas> -"precision" "real" "serial" "serial4" "serial8" "sequences" 
"smallint" "text"
    Nicolas> -"time" "timestamp" "timestamptz" "timetz" "tsquery" "tsvector"
    Nicolas> -"txid_snapshot" "unknown" "uuid" "varbit" "varchar" "varying" 
"without"
    Nicolas> -"xml" "zone"
    Nicolas> + "bigint" "bigserial" "bigserial" "bit" "bool" "boolean" "box" 
"bytea" "char"
    Nicolas> +"character" "cidr" "circle" "date" "daterange" "decimal" "double" 
"float4"
    Nicolas> +"float8" "inet" "int" "int2" "int4" "int4range" "int8" 
"int8range" "integer"
    Nicolas> +"interval" "jsonb" "jsonpath" "line" "lseg" "macaddr" "macaddr8" 
"money"
    Nicolas> +"name" "numeric" "numrange" "oid" "path" "point" "polygon" 
"precision" "real"
    Nicolas> +"regclass" "regcollation" "regconfig" "regdictionary" 
"regnamespace "
    Nicolas> +"regoper" "regoperator" "regproc" "regprocedure" "regrole" 
"regtype"
    Nicolas> +"sequences" "serial" "serial4" "serial8" "smallint" "smallserial" 
"text"
    Nicolas> +"time" "timestamp" "timestamptz" "timetz" "tsquery" "tsrange" 
"tstzrange"
    Nicolas> +"tsvector" "txid_snapshot" "unknown" "uuid" "varbit" "varchar" 
"varying"
    Nicolas> +"without" "xml" "zone"
    Nicolas>  )))

You have 'bigserial' in there twice.

Robert
--