shithub: scc

Download patch

ref: 1ce079ecedd0b95c5c367572b5872a78d2a309ba
parent: a90c5a46440cc4c6527b9b2c28d7d03b4553d82c
author: Roberto E. Vargas Caballero <[email protected]>
date: Sun Oct 6 08:26:54 EDT 2019

[ld] Fix command line parsing

The new function nextarg() simplifies parsing the command line parameters
and beautify the code.

--- a/src/cmd/ld/ld.h
+++ b/src/cmd/ld/ld.h
@@ -7,6 +7,7 @@
 
 /* main.c */
 extern void error(char *fmt, ...);
+extern char *nextarg(char **argp, char ***argv);
 
 /* symbol.c */
 extern int hasref(char *name);
--- a/src/cmd/ld/main.c
+++ b/src/cmd/ld/main.c
@@ -15,6 +15,7 @@
 int rflag;        /* preserve relocation bits */
 int dflag;        /* define common even with rflag */
 int gflag;        /* preserve debug symbols */
+int nmagic;       /* nmagic output */
 char *Dflag;      /* size of data */
 
 char *filename, *membname;
@@ -92,19 +93,37 @@
 	*bp = path;
 }
 
+char *
+nextarg(char **argp, char ***argv)
+{
+	char *ap = *argp, **av = *argv;
+
+	if (ap[1]) {
+		*argp += strlen(ap);
+		return ap+1;
+	}
+
+	if (av[1]) {
+		*argv = ++av;
+		return *av;
+	}
+
+	usage();
+}
+
 int
 main(int argc, char *argv[])
 {
 	int files = 0;
-	char *cp, *arg, **ap;
+	char *ap, **av;
 
-	for (ap = argv+1; *ap; ++ap) {
-		if (ap[0][0] != '-') {
+	for (av = argv+1; *av; ++av) {
+		if (av[0][0] != '-') {
 			files = 1;
 			continue;
 		}
-		for (cp = &ap[0][1]; *cp; ++cp) {
-			switch (*cp) {
+		for (ap = &av[0][1]; *ap; ++ap) {
+			switch (*ap) {
 			case 's':
 				sflag = 1;
 				break;
@@ -114,6 +133,7 @@
 			case 'X':
 				Xflag = 1;
 				break;
+			case 'i':
 			case 'r':
 				rflag = 1;
 				break;
@@ -120,49 +140,29 @@
 			case 'd':
 				dflag = 1;
 				break;
-			case 'i':
 			case 'n':
-				/* TODO */
+				nmagic = 1;
 				break;
-			case 'u':
 			case 'l':
-				/* FIXME: This way of handling options is wrong */
-				arg = (cp[1]) ? cp+1 : *++ap;
-				if (!arg)
-					goto usage;
-				goto next_arg;
+			case 'u':
+				nextarg(&ap, &av);
+				break;
 			case 'L':
-				arg = (cp[1]) ? cp+1 : *++ap;
-				if (!arg)
-					goto usage;
-				Lpath(arg);
-				goto next_arg;
+				Lpath(nextarg(&ap, &av));
+				break;
 			case 'o':
-				arg = (cp[1]) ? cp+1 : *++ap;
-				if (!arg)
-					goto usage;
-				output = arg;
-				goto next_arg;
+				output = nextarg(&ap, &av);
+				break;
 			case 'e':
-				arg = (cp[1]) ? cp+1 : *++ap;
-				if (!arg)
-					goto usage;
-				entry = arg;
-				goto next_arg;
+				entry = nextarg(&ap, &av);
+				break;
 			case 'D':
-				arg = (cp[1]) ? cp+1 : *++ap;
-				if (!arg)
-					goto usage;
-				Dflag = arg;
-				goto next_arg;
+				Dflag = nextarg(&ap, &av);
+				break;
 			default:
-			usage:
 				usage();
 			}
 		}
-
-		next_arg:
-			continue;
 	}
 
 	if (!files)
--- a/src/cmd/ld/pass1.c
+++ b/src/cmd/ld/pass1.c
@@ -274,25 +274,21 @@
 void
 pass1(int argc, char *argv[])
 {
-	char **ap, *cp, *arg;
+	char **av, *ap;
 
-	for (ap = argv+1; *ap; ++ap) {
-		if (ap[0][0] != '-') {
-			process(*ap);
+	for (av = argv+1; *av; ++av) {
+		if (av[0][0] != '-') {
+			process(*av);
 			continue;
 		}
-		for (cp = &ap[0][1]; *cp; ++cp) {
-			switch (*cp) {
+		for (ap = &av[0][1]; *ap; ++ap) {
+			switch (*ap) {
 			case 'l':
-				/* FIXME: we proccess arg again after this */
-				arg = (cp[1]) ? cp+1 : *++ap;
-				process(arg);
-				continue;
+				process(nextarg(&ap, &av));
+				break;
 			case 'u':
-				/* FIXME: we proccess arg again after this */
-				arg = (cp[1]) ? cp+1 : *++ap;
-				lookupsym(arg);
-				continue;
+				lookupsym(nextarg(&ap, &av));
+				break;
 			}
 		}
 	}