[postgresql-schema] Drop locals the type handlers compute but never read

CH
18h ago
+0-17

In PostgresqlSchemaCodegen, several process*TypeProperty handlers declare locals that nothing reads afterwards:

  • integer handler: dataType, isUuid, exclusiveMaximum
  • decimal handler: dataType, dataFormat, exclusiveMinimum, exclusiveMaximum, and a Float min/max computation
  • string handler: dataFormat
  • boolean handler: an unused columnDataTypeArguments list

This deletes them. One file, 17 deletions, no additions.

No generated output changes: the postgresql-schema output for seven specs across seven option combinations (idAutoIncEnabled, jsonDataType, identifierNamingConvention, defaultDatabaseName, namedParametersEnabled) is byte-for-byte identical before and after, and the committed sample regenerates without a diff.

First of a small series on this file; follow-ups extract the shared prologue, enum block and epilogue of these handlers into private helpers.

PR checklist

  • Read the contribution guidelines.
  • Build the project and update samples: not applicable, no generated output changes.
  • @mention the technical committee: no PostgreSQL schema owner is listed.

Summary by cubic

Removes unused local variables from the process*TypeProperty handlers in PostgresqlSchemaCodegen so they no longer compute values nothing reads.

  • Drops dataType, isUuid, and exclusiveMaximum from the integer handler, dataType, dataFormat, exclusiveMinimum, exclusiveMaximum, and a Float min/max computation from the decimal handler, dataFormat from the string handler, and an allocated argument list from the boolean handler.
  • No generated output changes; the postgresql-schema output for seven specs across seven option combinations is byte-for-byte identical before and after.