On Mon, Oct 28, 2024, at 10:30, Joel Jacobson wrote: > On Mon, Oct 28, 2024, at 08:56, jian he wrote: >> /* Check force_quote */ >> - if (!opts_out->csv_mode && (opts_out->force_quote || >> opts_out->force_quote_all)) >> + if (opts_out->format != COPY_FORMAT_CSV && (opts_out->force_quote || >> + opts_out->force_quote_all)) >> ereport(ERROR, >> (errcode(ERRCODE_FEATURE_NOT_SUPPORTED), >> /*- translator: %s is the name of a COPY option, e.g. ON_ERROR */ >> >> maybe this has a code indentation issue. >> since "if" and "opts_out" in the same column position. > > Thanks for review. > > I've fixed the indentation issues.
I've now installed pgindent, and will use it from hereon, to avoid this class of problems. New version where all three patches are now indented using pgindent. /Joel
v15-0001-Introduce-CopyFormat-and-replace-csv_mode-and-binary.patch
Description: Binary data
v15-0002-Add-raw-format-to-COPY-command.patch
Description: Binary data
v15-0003-Reorganize-option-validations.patch
Description: Binary data
