Changeset 3217 in kBuild for trunk/src/kmk

Timestamp:

Mar 30, 2018 10:11:15 PM (7 years ago)

Author:

bird

Message:

kmk_cmp: changed to use getopt_r.

File:

• trunk/src/kmk/kmkbuiltin/cmp.c (modified) (6 diffs)

trunk/src/kmk/kmkbuiltin/cmp.c

@@ -35 +35 @@
 __RCSID("$NetBSD: cmp.c,v 1.15 2006/01/19 20:44:57 garbled Exp $"); */
 
+/*********************************************************************************************************************************
+*   Header Files                                                                                                                 *
+*********************************************************************************************************************************/
+#define FAKES_NO_GETOPT_H /* bird */
 #ifdef _MSC_VER
 # define MSC_DO_64_BIT_IO /* for correct off_t */
@@ -51 +55 @@
 # include "mscfakes.h"
 #endif
-#include "getopt.h"
+#include "getopt_r.h"
 #include "kmkbuiltin.h"
 #include "cmp_extern.h"
 
 
+/*********************************************************************************************************************************
+*   Global Variables                                                                                                             *
+*********************************************************************************************************************************/
 static const struct option long_options[] =
 {
@@ -69 +76 @@
 kmk_builtin_cmp(int argc, char **argv, char **envp, PKMKBUILTINCTX pCtx)
 {
+    struct getopt_state_r gos;
     off_t skip1 = 0, skip2 = 0;
     int lflag = 0, sflag = 0;
@@ -74 +82 @@
     char *file1, *file2;
 
-#ifdef KMK_BUILTIN_STANDALONE
-    setlocale(LC_ALL, "");
-#endif
-
-    /* reset getopt and set progname. */
-    opterr = 1;
-    optarg = NULL;
-    optopt = 0;
-    optind = 0; /* init */
-
-    while ((ch = getopt_long(argc, argv, "ls", long_options, NULL)) != -1)
+    getopt_initialize_r(&gos, argc, argv, "ls", long_options, envp, pCtx);
+    while ((ch = getopt_long_r(&gos, NULL)) != -1)
     {
         switch (ch)
@@ -104 +103 @@
         }
     }
-    argv += optind;
-    argc -= optind;
+    argv += gos.optind;
+    argc -= gos.optind;
 
     if (argc < 2 || argc > 4)
@@ -148 +147 @@
 {
     KMKBUILTINCTX Ctx = { "kmk_cmp", NULL };
+    setlocale(LC_ALL, "");
     return kmk_builtin_cmp(argc, argv, envp, &Ctx);
 }