Re: [9front] rc: null list in concatenation line numbers

[cinap_lenrek@felloff.net]

+		if(f|l != 0){
+			emitf(Xloc);
+			emiti(f|l);
+		}

you need parentesis there around: f|l

+		/*
+		 * These are small enough that with a ton of sourcing,
+		 * overflow is a realistic risk. If we run out of files,
+		 * set to file 0, which is always '???'
+		 */
+		f = (t->file > LFMAX) ? 0 : t->file << LSHIFT;
+		l = (t->line > LMASK) ? -1 : t->line;

setting line to -1 also implicitely makes the file all ones...

+	if(i == nlexpath){
+		if(nlexpath < LFMAX){
+			lexpath = erealloc(lexpath, (nlexpath + 1)*sizeof(char*));
+			lexpath[nlexpath] = estrdup(zero);
+			lexfile = nlexpath++;
+		}else
+			lexfile = 0;
+	}

might want to use doubling the size for each realloc...

also wonder if the file can even change in the tree...
the file would only change when we source a file, no?

so i wonder if we need to keep track of the file in
the tree at all, and not just update it at runtime
in the thread.

--
cinap