Make USING and WITH optional to bring the syntax of \copy into exact

agreement with what the backend grammar actually accepts (which is a
bit looser than what its documentation claims).  Per report from Bill
Moran, though I did not use his patch since it removed all the
undocumented flexibility that the code historically had and the backend
still has.

diff --git a/src/bin/psql/copy.c b/src/bin/psql/copy.c
index c2f89550c7ff8e0b8d161dfa842101222eaf84a4..9766bece96d9b1a2ecdac55f44abd0455aa5bc56 100644 (file)
--- a/src/bin/psql/copy.c
+++ b/src/bin/psql/copy.c
@@ -3,7 +3,7 @@
  *
  * Copyright (c) 2000-2003, PostgreSQL Global Development Group
  *
- * $PostgreSQL: pgsql/src/bin/psql/copy.c,v 1.40 2004/01/26 22:35:32 tgl Exp $
+ * $PostgreSQL: pgsql/src/bin/psql/copy.c,v 1.41 2004/01/28 22:14:04 tgl Exp $
  */
 #include "postgres_fe.h"
 #include "copy.h"
@@ -36,11 +36,19 @@
  * parse_slash_copy
  * -- parses \copy command line
  *
- * Accepted syntax: \copy table [(columnlist)] [with oids] from|to filename [with ] [ oids ] [ delimiter char] [ null as string ]
+ * The documented preferred syntax is:
+ * \copy tablename [(columnlist)] from|to filename
+ *     [ with ] [ oids ] [ delimiter [as] char ] [ null [as] string ]
  * (binary is not here yet)
  *
- * Old syntax for backward compatibility: (2002-06-19):
- * \copy table [(columnlist)] [with oids] from|to filename [ using delimiters char] [ with null as string ]
+ * The pre-7.3 syntax was:
+ * \copy tablename [(columnlist)] [with oids] from|to filename
+ *     [ [using] delimiters char ] [ with null as string ]
+ *
+ * The actual accepted syntax is a rather unholy combination of these,
+ * plus some undocumented flexibility (for instance, the clauses after
+ * WITH can appear in any order).  The accepted syntax matches what
+ * the backend grammar actually accepts (see backend/parser/gram.y).
  *
  * table name can be double-quoted and can have a schema part.
  * column names can be double-quoted.
@@ -243,6 +251,9 @@ parse_slash_copy(const char *args)
                        0, false, pset.encoding);
        if (!(token && strcasecmp(token, "delimiters") == 0))
            goto error;
+   }
+   if (token && strcasecmp(token, "delimiters") == 0)
+   {
        token = strtokx(NULL, whitespace, NULL, "'",
                        '\\', false, pset.encoding);
        if (!token)
@@ -254,12 +265,22 @@ parse_slash_copy(const char *args)
 
    if (token)
    {
-       if (strcasecmp(token, "with") != 0)
-           goto error;
-       while ((token = strtokx(NULL, whitespace, NULL, NULL,
-                               0, false, pset.encoding)) != NULL)
+       /*
+        * WITH is optional.  Also, the backend will allow WITH followed by
+        * nothing, so we do too.
+        */
+       if (strcasecmp(token, "with") == 0)
+           token = strtokx(NULL, whitespace, NULL, NULL,
+                           0, false, pset.encoding);
+
+       while (token)
        {
-           if (strcasecmp(token, "delimiter") == 0)
+           /* someday allow BINARY here */
+           if (strcasecmp(token, "oids") == 0)
+           {
+               result->oids = true;
+           }
+           else if (strcasecmp(token, "delimiter") == 0)
            {
                token = strtokx(NULL, whitespace, NULL, "'",
                                '\\', false, pset.encoding);
@@ -285,6 +306,9 @@ parse_slash_copy(const char *args)
            }
            else
                goto error;
+
+           token = strtokx(NULL, whitespace, NULL, NULL,
+                           0, false, pset.encoding);
        }
    }