From: Tom Lane <tgl@sss.pgh.pa.us>
Date: Thu, 8 May 2014 16:42:56 +0000 (-0400)
Subject: Fix comment.
X-Git-Tag: REL9_4_BETA1~27
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=1e81f8462aceb07d520daaabdc999b879381bd12;p=postgresql

Fix comment.

Previous commit was confused about the case we're handling: actually,
what the patch is dealing with is platforms that have optreset, *and*
have <getopt.h>, but the latter fails to declare the former.  Because
we use a linking probe to set HAVE_INT_OPTRESET, we need to be sure we
have a declaration even if <getopt.h> doesn't think it exists.
---

diff --git a/src/include/pg_getopt.h b/src/include/pg_getopt.h
index 0b38404ed3..531c265f7c 100644
--- a/src/include/pg_getopt.h
+++ b/src/include/pg_getopt.h
@@ -32,8 +32,8 @@ extern int	optopt;
 #endif   /* HAVE_GETOPT_H */
 
 /*
- * Some platforms have optreset but not <getopt.h>.  Cygwin, however,
- * doesn't like this either.
+ * Some platforms have optreset but fail to declare it in <getopt.h>, so cope.
+ * Cygwin, however, doesn't like this either.
  */
 #if defined(HAVE_INT_OPTRESET) && !defined(__CYGWIN__)
 extern int	optreset;