git: 0f4467ce444b - main - cp: Move the flags around a bit.

From: Dag-Erling Smørgrav <des_at_FreeBSD.org>
Date: Thu, 14 Dec 2023 16:07:29 UTC
The branch main has been updated by des:

URL: https://cgit.FreeBSD.org/src/commit/?id=0f4467ce444b201468d2268958130f495951ca3c

commit 0f4467ce444b201468d2268958130f495951ca3c
Author:     Dag-Erling Smørgrav <des@FreeBSD.org>
AuthorDate: 2023-12-14 16:07:00 +0000
Commit:     Dag-Erling Smørgrav <des@FreeBSD.org>
CommitDate: 2023-12-14 16:07:00 +0000

    cp: Move the flags around a bit.
    
    - The HLPR flags are grouped together at the beginning because they are
      the standard flags for programs using FTS.  Move the N flag out from
      among them to its correct place in the sequence.
    - The Pflag variable isn't used outside main(), but moving it out lets
      us skip initialization and keeps it with its friends H, L and R.
    
    MFC after:      1 week
    Sponsored by:   Klara, Inc.
    Reviewed by:    kevans
    Differential Revision:  https://reviews.freebsd.org/D43063
---
 bin/cp/cp.c | 13 ++++++-------
 1 file changed, 6 insertions(+), 7 deletions(-)

diff --git a/bin/cp/cp.c b/bin/cp/cp.c
index 78ded7af3d5a..1455979bdb6e 100644
--- a/bin/cp/cp.c
+++ b/bin/cp/cp.c
@@ -73,7 +73,7 @@ static char emptystring[] = "";
 PATH_T to = { to.p_path, emptystring, "" };
 
 int Nflag, fflag, iflag, lflag, nflag, pflag, sflag, vflag;
-static int Hflag, Lflag, Rflag, rflag;
+static int Hflag, Lflag, Pflag, Rflag, rflag;
 volatile sig_atomic_t info;
 
 enum op { FILE_TO_FILE, FILE_TO_DIR, DIR_TO_DNE };
@@ -86,12 +86,11 @@ main(int argc, char *argv[])
 {
 	struct stat to_stat, tmp_stat;
 	enum op type;
-	int Pflag, ch, fts_options, r, have_trailing_slash;
+	int ch, fts_options, r, have_trailing_slash;
 	char *target;
 
 	fts_options = FTS_NOCHDIR | FTS_PHYSICAL;
-	Pflag = 0;
-	while ((ch = getopt(argc, argv, "HLNPRafilnprsvx")) != -1)
+	while ((ch = getopt(argc, argv, "HLPRafilNnprsvx")) != -1)
 		switch (ch) {
 		case 'H':
 			Hflag = 1;
@@ -101,9 +100,6 @@ main(int argc, char *argv[])
 			Lflag = 1;
 			Hflag = Pflag = 0;
 			break;
-		case 'N':
-			Nflag = 1;
-			break;
 		case 'P':
 			Pflag = 1;
 			Hflag = Lflag = 0;
@@ -128,6 +124,9 @@ main(int argc, char *argv[])
 		case 'l':
 			lflag = 1;
 			break;
+		case 'N':
+			Nflag = 1;
+			break;
 		case 'n':
 			nflag = 1;
 			fflag = iflag = 0;