Remove obsolete comment in CopyReadLineText().
authorTom Lane <tgl@sss.pgh.pa.us>
Sat, 6 Apr 2024 15:16:27 +0000 (11:16 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Sat, 6 Apr 2024 15:16:27 +0000 (11:16 -0400)
When this bit of commentary was written, it was alluding to the
fact that we looked for newlines and EOD markers in the raw
(not yet encoding-converted) input data.  We don't do that anymore,
preferring to batch the conversion of larger chunks of input and
split it into lines later.  Hence there's no longer any need for
assumptions about the relevant characters being encoding-invariant,
and we should remove this comment saying we assume that.

Discussion: https://postgr.es/m/1461688.1712347668@sss.pgh.pa.us

src/backend/commands/copyfromparse.c

index 7ddd27f5c6efc67ebd10e67c322e359d1fbc2ea5..7efcb891598f05a4e936502bf2990aa9c71b4d55 100644 (file)
@@ -1206,9 +1206,6 @@ CopyReadLineText(CopyFromState cstate)
     * value and are put in line_buf.  We keep just enough state to know if we
     * are currently in a quoted field or not.
     *
-    * These four characters, and the CSV escape and quote characters, are
-    * assumed the same in frontend and backend encodings.
-    *
     * The input has already been converted to the database encoding.  All
     * supported server encodings have the property that all bytes in a
     * multi-byte sequence have the high bit set, so a multibyte character