The result of getopt() should be compared to -1, not EOF, per

pgsql-hackers discussion of this date.

diff --git a/contrib/dbase/dbf2pg.c b/contrib/dbase/dbf2pg.c
index 8b7d989a072767a14aced6d59f6517ea7001126d..5f5e9d2823f403c4b8138b9a763b9fcf6d4e571f 100644 (file)
--- a/contrib/dbase/dbf2pg.c
+++ b/contrib/dbase/dbf2pg.c
@@ -16,6 +16,9 @@
 #ifdef HAVE_ICONV_H
 #include 
 #endif
+#ifdef HAVE_GETOPT_H
+#include 
+#endif
 
 #include "libpq-fe.h"
 #include "dbf.h"
@@ -673,7 +676,7 @@ main(int argc, char **argv)
    char       *query;
    dbhead     *dbh;
 
-   while ((i = getopt(argc, argv, "DWflucvh:b:e:d:t:s:B:U:F:T:")) != EOF)
+   while ((i = getopt(argc, argv, "DWflucvh:b:e:d:t:s:B:U:F:T:")) != -1)
    {
        switch (i)
        {

diff --git a/contrib/oid2name/oid2name.c b/contrib/oid2name/oid2name.c
index 2d4e2b8d0f8e2c45a9f6c406c50aacddd8c529e3..dfd261240ac1d55b1dbcc8bfb4d1dee51343c7b3 100644 (file)
--- a/contrib/oid2name/oid2name.c
+++ b/contrib/oid2name/oid2name.c
@@ -67,7 +67,7 @@ get_opts(int argc, char **argv, struct options * my_opts)
    my_opts->remotepass = 0;
 
    /* get opts */
-   while ((c = getopt(argc, argv, "H:p:U:P:d:t:o:xh?")) != EOF)
+   while ((c = getopt(argc, argv, "H:p:U:P:d:t:o:xh?")) != -1)
    {
        switch (c)
        {

diff --git a/contrib/pgbench/pgbench.c b/contrib/pgbench/pgbench.c
index 8d20caf3e4c11cdc683f80abc5cb1b5e1cadc500..56d6a7128a61f452db30c6e8f67ee31bf9e58b25 100644 (file)
--- a/contrib/pgbench/pgbench.c
+++ b/contrib/pgbench/pgbench.c
@@ -1,5 +1,5 @@
 /*
- * $Header: /cvsroot/pgsql/contrib/pgbench/pgbench.c,v 1.13 2001/11/05 17:46:23 momjian Exp $
+ * $Header: /cvsroot/pgsql/contrib/pgbench/pgbench.c,v 1.14 2002/01/10 01:11:45 tgl Exp $
  *
  * pgbench: a simple TPC-B like benchmark program for PostgreSQL
  * written by Tatsuo Ishii
@@ -641,7 +641,7 @@ main(int argc, char **argv)
    PGconn     *con;
    PGresult   *res;
 
-   while ((c = getopt(argc, argv, "ih:nvp:dc:t:s:U:P:CS")) != EOF)
+   while ((c = getopt(argc, argv, "ih:nvp:dc:t:s:U:P:CS")) != -1)
    {
        switch (c)
        {

diff --git a/src/backend/bootstrap/bootstrap.c b/src/backend/bootstrap/bootstrap.c
index 076b31fb17e1bdf9ab8d915cb7c029b99ea4718d..cd118d30d74272cb5fd27d5bb4f5d318961c798c 100644 (file)
--- a/src/backend/bootstrap/bootstrap.c
+++ b/src/backend/bootstrap/bootstrap.c
@@ -8,7 +8,7 @@
  * Portions Copyright (c) 1994, Regents of the University of California
  *
  * IDENTIFICATION
- *   $Header: /cvsroot/pgsql/src/backend/bootstrap/bootstrap.c,v 1.119 2001/10/25 05:49:22 momjian Exp $
+ *   $Header: /cvsroot/pgsql/src/backend/bootstrap/bootstrap.c,v 1.120 2002/01/10 01:11:45 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -250,7 +250,7 @@ BootstrapMain(int argc, char *argv[])
                                                 * variable */
    }
 
-   while ((flag = getopt(argc, argv, "B:dD:Fo:px:")) != EOF)
+   while ((flag = getopt(argc, argv, "B:dD:Fo:px:")) != -1)
    {
        switch (flag)
        {

diff --git a/src/backend/postmaster/postmaster.c b/src/backend/postmaster/postmaster.c
index f248019d0abdcd885e7226c00cb08e905e832c35..c480bf2f41f60c60451bd2d5921a7ec9bf04fded 100644 (file)
--- a/src/backend/postmaster/postmaster.c
+++ b/src/backend/postmaster/postmaster.c
@@ -37,7 +37,7 @@
  *
  *
  * IDENTIFICATION
- *   $Header: /cvsroot/pgsql/src/backend/postmaster/postmaster.c,v 1.263 2002/01/06 21:40:02 tgl Exp $
+ *   $Header: /cvsroot/pgsql/src/backend/postmaster/postmaster.c,v 1.264 2002/01/10 01:11:45 tgl Exp $
  *
  * NOTES
  *
@@ -420,7 +420,7 @@ PostmasterMain(int argc, char *argv[])
     * with the wrong argument. Death and destruction will occur.
     */
    opterr = 1;
-   while ((opt = getopt(argc, argv, "A:a:B:b:c:D:d:Fh:ik:lm:MN:no:p:Ss-:")) != EOF)
+   while ((opt = getopt(argc, argv, "A:a:B:b:c:D:d:Fh:ik:lm:MN:no:p:Ss-:")) != -1)
    {
        switch (opt)
        {
@@ -458,7 +458,7 @@ PostmasterMain(int argc, char *argv[])
    optreset = 1;               /* some systems need this too */
 #endif
 
-   while ((opt = getopt(argc, argv, "A:a:B:b:c:D:d:Fh:ik:lm:MN:no:p:Ss-:")) != EOF)
+   while ((opt = getopt(argc, argv, "A:a:B:b:c:D:d:Fh:ik:lm:MN:no:p:Ss-:")) != -1)
    {
        switch (opt)
        {

diff --git a/src/backend/tcop/postgres.c b/src/backend/tcop/postgres.c
index 68ae795e6a7cdf664617e9ae187661ebcc60f77e..0609e1e9531cb97d17f6791d14509a2939ef09f5 100644 (file)
--- a/src/backend/tcop/postgres.c
+++ b/src/backend/tcop/postgres.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *   $Header: /cvsroot/pgsql/src/backend/tcop/postgres.c,v 1.244 2002/01/01 23:16:22 tgl Exp $
+ *   $Header: /cvsroot/pgsql/src/backend/tcop/postgres.c,v 1.245 2002/01/10 01:11:45 tgl Exp $
  *
  * NOTES
  *   this is the "main" module of the postgres backend and
@@ -1165,7 +1165,7 @@ PostgresMain(int argc, char *argv[], const char *username)
    secure = true;
    ctx = PGC_POSTMASTER;
 
-   while ((flag = getopt(argc, argv, "A:B:c:CD:d:Eef:FiNOPo:p:S:st:v:W:x:-:")) != EOF)
+   while ((flag = getopt(argc, argv, "A:B:c:CD:d:Eef:FiNOPo:p:S:st:v:W:x:-:")) != -1)
        switch (flag)
        {
            case 'A':
@@ -1626,7 +1626,7 @@ PostgresMain(int argc, char *argv[], const char *username)
    if (!IsUnderPostmaster)
    {
        puts("\nPOSTGRES backend interactive interface ");
-       puts("$Revision: 1.244 $ $Date: 2002/01/01 23:16:22 $\n");
+       puts("$Revision: 1.245 $ $Date: 2002/01/10 01:11:45 $\n");
    }
 
    /*

diff --git a/src/bin/pg_dump/pg_restore.c b/src/bin/pg_dump/pg_restore.c
index 6051dbf83b3216f1eb5b6d264f162e4cb4ba1f9d..ac740042f42b50bdce99c130d100706b6cc65e7f 100644 (file)
--- a/src/bin/pg_dump/pg_restore.c
+++ b/src/bin/pg_dump/pg_restore.c
@@ -34,7 +34,7 @@
  *
  *
  * IDENTIFICATION
- *     $Header: /cvsroot/pgsql/src/bin/pg_dump/pg_restore.c,v 1.31 2001/11/26 23:10:36 tgl Exp $
+ *     $Header: /cvsroot/pgsql/src/bin/pg_dump/pg_restore.c,v 1.32 2002/01/10 01:11:45 tgl Exp $
  *
  * Modifications - 28-Jun-2000 - [email protected]
  *
@@ -67,7 +67,6 @@
 #include 
 #endif
 
-/* FreeBSD 4.3 seems to put getopt in unistd.h */
 #include 
 
 #ifdef HAVE_GETOPT_H
@@ -164,7 +163,7 @@ main(int argc, char **argv)
    }
 
 #ifdef HAVE_GETOPT_LONG
-   while ((c = getopt_long(argc, argv, "acCd:f:F:h:i:lL:NoOp:P:rRsS:t:T:uU:vWxX:", cmdopts, NULL)) != EOF)
+   while ((c = getopt_long(argc, argv, "acCd:f:F:h:i:lL:NoOp:P:rRsS:t:T:uU:vWxX:", cmdopts, NULL)) != -1)
 #else
    while ((c = getopt(argc, argv, "acCd:f:F:h:i:lL:NoOp:P:rRsS:t:T:uU:vWxX:")) != -1)
 #endif

diff --git a/src/interfaces/ecpg/preproc/ecpg.c b/src/interfaces/ecpg/preproc/ecpg.c
index 5e809042155b03ead87fe9f90fd7abd6c38f80ec..b6761a6dee181c91dd6cef350cba01c04442f326 100644 (file)
--- a/src/interfaces/ecpg/preproc/ecpg.c
+++ b/src/interfaces/ecpg/preproc/ecpg.c
@@ -1,4 +1,4 @@
-/* $Header: /cvsroot/pgsql/src/interfaces/ecpg/preproc/ecpg.c,v 1.51 2001/10/28 06:26:11 momjian Exp $ */
+/* $Header: /cvsroot/pgsql/src/interfaces/ecpg/preproc/ecpg.c,v 1.52 2002/01/10 01:11:45 tgl Exp $ */
 
 /* New main for ecpg, the PostgreSQL embedded SQL precompiler. */
 /* (C) Michael Meskes  Feb 5th, 1998 */
@@ -7,9 +7,8 @@
 #include "postgres_fe.h"
 
 #include 
-
 #ifdef HAVE_GETOPT_H
-#include "getopt.h"
+#include 
 #endif
 
 extern int optind;
@@ -108,7 +107,7 @@ main(int argc, char *const argv[])
    add_include_path("/usr/local/include");
    add_include_path(".");
 
-   while ((c = getopt(argc, argv, "vo:I:tD:d")) != EOF)
+   while ((c = getopt(argc, argv, "vo:I:tD:d")) != -1)
    {
        switch (c)
        {