ranlib: Remove offset and oldsiz globals - scc - simple c99 compiler
HTML git clone git://git.simple-cc.org/scc
DIR Log
DIR Files
DIR Refs
DIR README
DIR LICENSE
---
DIR commit 334fc49dab766c427e4ea98719cbaf869e60b6c9
DIR parent 0dcd0d571ff51bbc87adcec9531f6a21707823d6
HTML Author: Roberto E. Vargas Caballero <k0ga@shike2.net>
Date: Mon, 11 May 2026 10:04:31 +0200
ranlib: Remove offset and oldsiz globals
How these variables were used was a bit weirdd, since they could
just be parameters passed to newsymbol(), and avoid the delta
calculation storing only offset without any index in the first
stage. That makes trivial to do the second phase.
Diffstat:
M src/cmd/scc-ranlib.c | 19 ++++++++-----------
1 file changed, 8 insertions(+), 11 deletions(-)
---
DIR diff --git a/src/cmd/scc-ranlib.c b/src/cmd/scc-ranlib.c
@@ -28,7 +28,6 @@ static long nsymbols;
static int status, artype, nolib;
static char *filename, *membname;
static Symdef *htab[NR_SYMDEF], *head;
-static long offset, oldsize;
char *argv0;
static void
@@ -85,7 +84,7 @@ lookup(char *name)
}
static int
-newsymbol(Symbol *sym)
+newsymbol(Symbol *sym, long off)
{
Symdef *np;
@@ -103,7 +102,7 @@ newsymbol(Symbol *sym)
break;
case 'U':
np->type = sym->type;
- np->offset = offset;
+ np->offset = off;
break;
default:
if (sym->type != 'C') {
@@ -139,8 +138,6 @@ newmember(FILE *fp, long off)
Obj *obj;
Symbol sym;
- offset = off;
-
t = objprobe(fp, NULL);
if (t == -1 || artype != -1 && artype != t) {
nolib = 1;
@@ -160,7 +157,7 @@ newmember(FILE *fp, long off)
}
for (i = 0; getsym(obj, &i, &sym); i++) {
- if (!newsymbol(&sym))
+ if (!newsymbol(&sym, off))
goto error;
}
@@ -174,13 +171,13 @@ error:
static int
readsyms(FILE *fp)
{
- long cur, off;
+ long idxsiz, cur, off;
char memb[SARNAM+1];
nolib = 0;
artype = -1;
nsymbols = 0;
- oldsize = 0;
+ idxsiz = 0;
if (!archive(fp)) {
error("file format not recognized");
@@ -192,7 +189,7 @@ readsyms(FILE *fp)
goto corrupted;
if (strcmp(memb, "/") == 0 || strcmp(memb, "__.SYMDEF") == 0) {
- oldsize = off;
+ idxsiz = off;
cur += off;
}
@@ -208,7 +205,7 @@ readsyms(FILE *fp)
default:
membname = memb;
if (objprobe(fp, NULL) != -1)
- newmember(fp, cur);
+ newmember(fp, cur - idxsiz);
membname = NULL;
fseek(fp, cur, SEEK_SET);
fseek(fp, off, SEEK_CUR);
@@ -316,7 +313,7 @@ ranlib(char *fname)
size = sizeof(struct ar_hdr) + prop.size + (prop.size & 1);
for (i = 0; i < nsymbols; i++)
- offs[i] += size - oldsize;
+ offs[i] += size;
/* Second pass: rewrite index with corrected offsets */
rewind(idx);