Remove text run SCBlock offset

The \newpara change (commit 7f2d0abd) removed the only situation when the
start of a run might not coincide with the start of an SCBlock, or vice-versa
(although not all SCBlocks correspond to text runs, obviously).

Therefore, the offset is always zero and can be removed, simplifying the code.

5 files changed, 33 insertions, 105 deletions

diff --git a/src/debugger.c b/src/debugger.c
index 39a5cb4..c5aaa1f 100644
--- a/src/debugger.c
+++ b/src/debugger.c
@@ -45,7 +45,6 @@ struct run_debug
 	int np;
 	size_t len;
 	void *scblock; /* Don't you dare try to dereference this */
-	size_t offs;
 	size_t para_offs;
 };
 
@@ -117,14 +116,14 @@ static void debug_text_para(Paragraph *para, cairo_t *cr, double *ypos,
 	plot_text(cr, ypos, fontdesc, tmp);
 
 	for ( i=0; i<nrun; i++ ) {
-		size_t scblock_offs, para_offs, len;
+		size_t para_offs, len;
 		SCBlock *scblock;
-		if ( para_debug_run_info(para, i, &len, &scblock, &scblock_offs, &para_offs) ) {
+		if ( para_debug_run_info(para, i, &len, &scblock, &para_offs) ) {
 			plot_text(cr, ypos, fontdesc, "Error");
 		} else {
 
-			snprintf(tmp, 255, "  Run %i: len %li, SCBlock %p offs %li, para offs %li",
-			         i, len, scblock, scblock_offs, para_offs);
+			snprintf(tmp, 255, "  Run %i: len %li, SCBlock %p, para offs %li",
+			         i, len, scblock, para_offs);
 			plot_text(cr, ypos, fontdesc, tmp);
 			if ( len != rd[*dpos].len ) {
 				snprintf(tmp, 255, "   (len was %li)", rd[*dpos].len);
@@ -136,11 +135,6 @@ static void debug_text_para(Paragraph *para, cairo_t *cr, double *ypos,
 				plot_text(cr, ypos, fontdesc, tmp);
 				(*changesig) += i*(*dpos)*para_offs*(*ypos);
 			}
-			if ( scblock_offs != rd[*dpos].offs ) {
-				snprintf(tmp, 255, "   (offs was %li)", rd[*dpos].offs);
-				plot_text(cr, ypos, fontdesc, tmp);
-				(*changesig) += i*(*dpos)*scblock_offs*(*ypos);
-			}
 			(*dpos)++;
 
 		}
@@ -172,11 +166,11 @@ static void record_runs(struct debugwindow *dbgw)
 
 		for ( j=0; j<nrun; j++ ) {
 
-			size_t scblock_offs, para_offs, len;
+			size_t para_offs, len;
 			SCBlock *scblock;
 
 			if ( para_debug_run_info(para, j, &len, &scblock,
-			                         &scblock_offs, &para_offs) )
+			                         &para_offs) )
 			{
 				continue;
 			}
@@ -184,7 +178,6 @@ static void record_runs(struct debugwindow *dbgw)
 			dbgw->runs[n].np = i;
 			dbgw->runs[n].len = len;
 			dbgw->runs[n].scblock = scblock;
-			dbgw->runs[n].offs = scblock_offs;
 			dbgw->runs[n].para_offs = para_offs;
 			n++;
 
diff --git a/src/frame.c b/src/frame.c
index 7370412..8776fbd 100644
--- a/src/frame.c
+++ b/src/frame.c
@@ -42,7 +42,6 @@ struct text_run
 	SCBlock              *macro_real_block;
 	SCBlock              *macro_contents;
 	int                   macro_editable;
-	size_t                scblock_offs_bytes;  /* Offset from start of SCBlock */
 	size_t                para_offs_bytes;  /* Offset from start of paragraph */
 	size_t                len_bytes;
 	PangoFontDescription *fontdesc;
@@ -322,8 +321,7 @@ void wrap_paragraph(Paragraph *para, PangoContext *pc, double w,
 		const char *run_text;
 		guint16 r, g, b;
 
-		run_text = sc_block_contents(para->runs[i].scblock)
-		           + para->runs[i].scblock_offs_bytes;
+		run_text = sc_block_contents(para->runs[i].scblock);
 
 		attr = pango_attr_font_desc_new(para->runs[i].fontdesc);
 		attr->start_index = pos;
@@ -380,7 +378,7 @@ void set_newline_at_end(Paragraph *para, SCBlock *bl)
 
 void add_run(Paragraph *para, SCBlock *scblock,
              SCBlock *macro_real, SCBlock *contents_top,
-             size_t offs_bytes, size_t len_bytes, PangoFontDescription *fdesc,
+             size_t len_bytes, PangoFontDescription *fdesc,
              double col[4], int macro_editable)
 {
 	struct text_run *runs_new;
@@ -402,7 +400,6 @@ void add_run(Paragraph *para, SCBlock *scblock,
 	para->runs[para->n_runs].macro_real_block = macro_real;
 	para->runs[para->n_runs].macro_contents = contents_top;
 	para->runs[para->n_runs].macro_editable = macro_editable;
-	para->runs[para->n_runs].scblock_offs_bytes = offs_bytes;
 	para->runs[para->n_runs].para_offs_bytes = para->offset_last;
 	para->offset_last += len_bytes;
 	para->runs[para->n_runs].len_bytes = len_bytes;
@@ -954,7 +951,7 @@ size_t pos_trail_to_offset(Paragraph *para, size_t offs, int trail)
 	}
 
 	/* Get the text for the run */
-	run_text = sc_block_contents(run->scblock) + run->scblock_offs_bytes;
+	run_text = sc_block_contents(run->scblock);
 
 	/* Turn  the paragraph offset into a run offset */
 	run_offs = offs - run->para_offs_bytes;
@@ -979,7 +976,7 @@ void insert_text_in_paragraph(Paragraph *para, size_t offs, const char *t)
 	int nrun;
 	int i;
 	struct text_run *run;
-	size_t run_offs, scblock_offs, ins_len;
+	size_t run_offs, ins_len;
 
 	/* Find which run we are in */
 	nrun = which_run(para, offs);
@@ -1019,8 +1016,7 @@ void insert_text_in_paragraph(Paragraph *para, size_t offs, const char *t)
 
 	/* Translate paragraph offset for insertion into SCBlock offset */
 	run_offs = offs - run->para_offs_bytes;
-	scblock_offs = run_offs + run->scblock_offs_bytes;
-	sc_insert_text(run->scblock, scblock_offs, t);
+	sc_insert_text(run->scblock, run_offs, t);
 
 	/* Update length of this run */
 	ins_len = strlen(t);
@@ -1028,37 +1024,11 @@ void insert_text_in_paragraph(Paragraph *para, size_t offs, const char *t)
 
 	/* Update offsets of subsequent runs */
 	for ( i=nrun+1; i<para->n_runs; i++ ) {
-		if ( para->runs[i].scblock == run->scblock ) {
-			para->runs[i].scblock_offs_bytes += ins_len;
-		}
 		para->runs[i].para_offs_bytes += ins_len;
 	}
 }
 
 
-static void fixup_subsq(int first_para, struct frame *fr, SCBlock *scblock,
-                        size_t del_len)
-{
-	int i;
-
-	for ( i=first_para; i<fr->n_paras; i++ ) {
-		int j;
-		int done = 0;
-		Paragraph *para = fr->paras[i];
-		if ( para->type != PARA_TYPE_TEXT ) continue;
-		for ( j=0; j<para->n_runs; j++ ) {
-			if ( para->runs[j].scblock != scblock) {
-				done = 1;
-				break;
-				}
-			printf("subsq para %p run %i del %i\n", para, i, (int)del_len);
-			para->runs[j].scblock_offs_bytes -= del_len;
-		}
-		if ( done ) break;
-	}
-}
-
-
 static void delete_text_paragraph(Paragraph *para, int p, struct frame *fr)
 {
 	int i;
@@ -1084,23 +1054,15 @@ static void delete_text_paragraph(Paragraph *para, int p, struct frame *fr)
 		}
 
 		/* Delete from the corresponding SC block */
-		scblock_delete_text(run->scblock, run->scblock_offs_bytes,
-		                    run->scblock_offs_bytes + run->len_bytes);
+		scblock_delete_text(run->scblock, 0, run->len_bytes);
 
 		/* Fix up the offsets of the subsequent text runs */
 		size_t del_len = run->len_bytes;
 		run->len_bytes -= del_len;
 		for ( j=i+1; j<para->n_runs; j++ ) {
-			if ( para->runs[j].scblock == run->scblock ) {
-				para->runs[j].scblock_offs_bytes -= del_len;
-			}
 			para->runs[j].para_offs_bytes -= del_len;
 		}
 
-		/* ... and in subsequent paragraphs, if they're from the same
-		 * SCBlock */
-		fixup_subsq(p+1, fr, run->scblock, del_len);
-
 	}
 }
 
@@ -1234,8 +1196,8 @@ size_t delete_text_in_paragraph(struct frame *fr, int npara, size_t offs1, ssize
 		}
 
 		/* Delete from the corresponding SC block */
-		scblock_offs1 = ds + run->scblock_offs_bytes;
-		scblock_offs2 = de + run->scblock_offs_bytes;
+		scblock_offs1 = ds;
+		scblock_offs2 = de;
 		sum_del += scblock_offs2 - scblock_offs1;
 		scblock_delete_text(run->scblock, scblock_offs1, scblock_offs2);
 
@@ -1243,17 +1205,9 @@ size_t delete_text_in_paragraph(struct frame *fr, int npara, size_t offs1, ssize
 		size_t del_len = de - ds;
 		run->len_bytes -= del_len;
 		for ( i=nrun+1; i<para->n_runs; i++ ) {
-			if ( para->runs[i].scblock == run->scblock ) {
-				printf("para %p run %i del %i\n", para, i, (int)del_len);
-				para->runs[i].scblock_offs_bytes -= del_len;
-			}
 			para->runs[i].para_offs_bytes -= del_len;
 		}
 
-		/* ... and in subsequent paragraphs, if they're from the same
-		 * SCBlock */
-		fixup_subsq(npara+1, fr, run->scblock, del_len);
-
 		offs1 -= del_len;
 		offs2 -= del_len;
 
@@ -1265,8 +1219,7 @@ size_t delete_text_in_paragraph(struct frame *fr, int npara, size_t offs1, ssize
 
 static char *run_text(struct text_run *run)
 {
-	return strndup(sc_block_contents(run->scblock)+run->scblock_offs_bytes,
-	               run->len_bytes);
+	return strndup(sc_block_contents(run->scblock), run->len_bytes);
 }
 
 
@@ -1280,11 +1233,10 @@ void show_para(Paragraph *p)
 		printf("%i runs:\n", p->n_runs);
 		for ( i=0; i<p->n_runs; i++ ) {
 			char *tmp = run_text(&p->runs[i]);
-			printf("  Run %2i: para offs %lli, SCBlock %p offs %lli, len "
+			printf("  Run %2i: para offs %lli, SCBlock %p, len "
 			       "%lli %s '%s'\n",
 			       i, (long long int)p->runs[i].para_offs_bytes,
 			       p->runs[i].scblock,
-			       (long long int)p->runs[i].scblock_offs_bytes,
 			       (long long int)p->runs[i].len_bytes,
 			       pango_font_description_to_string(p->runs[i].fontdesc),
 			       tmp);
@@ -1374,12 +1326,10 @@ static void check_para(Paragraph *para)
 	for ( i=0; i<para->n_runs; i++ ) {
 		const char *run_text;
 		if ( sc_block_contents(para->runs[i].scblock) == NULL ) continue;
-		run_text = sc_block_contents(para->runs[i].scblock)
-		           + para->runs[i].scblock_offs_bytes;
+		run_text = sc_block_contents(para->runs[i].scblock);
 		if ( strlen(run_text) < para->runs[i].len_bytes ) {
 			printf("found a wrong run\n");
-			printf("run %i sc offs %li, para offs %li text '%s'\n", i,
-			       (long int)para->runs[i].scblock_offs_bytes,
+			printf("run %i, para offs %li text '%s'\n", i,
 			       (long int)para->runs[i].para_offs_bytes,
 			       run_text);
 			printf("(the SCBlock contains '%s'\n", sc_block_contents(para->runs[i].scblock));
@@ -1436,7 +1386,6 @@ static SCBlock *split_text_paragraph(struct frame *fr, int pn, size_t pos,
 			pnew->runs[0].para_offs_bytes = 0;
 			pnew->runs[0].scblock = rr->scblock;
 			pnew->runs[0].macro_real_block = rr->macro_real_block;
-			pnew->runs[0].scblock_offs_bytes = rr->scblock_offs_bytes + run_offs;
 			pnew->runs[0].para_offs_bytes = 0;
 			pnew->runs[0].len_bytes = 0;
 			pnew->runs[0].fontdesc = pango_font_description_copy(rr->fontdesc);
@@ -1464,7 +1413,6 @@ static SCBlock *split_text_paragraph(struct frame *fr, int pn, size_t pos,
 		/* First run of the new paragraph contains the leftover text */
 		pnew->runs[0].scblock = rr->scblock;
 		pnew->runs[0].macro_real_block = rr->macro_real_block;
-		pnew->runs[0].scblock_offs_bytes = rr->scblock_offs_bytes + run_offs;
 		pnew->runs[0].para_offs_bytes = 0;
 		pnew->runs[0].len_bytes = rr->len_bytes - run_offs;
 		printf("%i - %i\n", (int)rr->len_bytes, (int)run_offs);
@@ -1498,18 +1446,14 @@ static SCBlock *split_text_paragraph(struct frame *fr, int pn, size_t pos,
 	/* If the first and second paragraphs have the same SCBlock, split it */
 	if ( (rr->scblock != NULL) && (rr->scblock == pnew->runs[0].scblock) ) {
 
-		size_t sc_offs;
-		sc_offs = rr->scblock_offs_bytes + run_offs;
-		printf("splitting SCBlock at %i\n", (int)sc_offs);
+		printf("splitting SCBlock at %i\n", (int)run_offs);
 		printf("old block: '%s'\n", sc_block_contents(rr->scblock));
-		pnew->runs[0].scblock = sc_block_split(rr->scblock, sc_offs);
-		pnew->runs[0].scblock_offs_bytes = 0;
+		pnew->runs[0].scblock = sc_block_split(rr->scblock, run_offs);
 		printf("new block 1: '%s'\n", sc_block_contents(rr->scblock));
 		printf("new block 2: '%s'\n", sc_block_contents(pnew->runs[0].scblock));
 		printf("run %p block %p\n", &pnew->runs[0], pnew->runs[0].scblock);
 
-		printf("run %i sc offs %li, para offs %li\n", 0,
-		       (long int)pnew->runs[0].scblock_offs_bytes,
+		printf("run %i, para offs %li\n", 0,
 		       (long int)pnew->runs[0].para_offs_bytes);
 		printf("len %i\n", (int)pnew->runs[0].len_bytes);
 	}
@@ -1571,7 +1515,7 @@ int get_sc_pos(struct frame *fr, int pn, size_t pos,
 	}
 	run = &para->runs[nrun];
 
-	*ppos = run->scblock_offs_bytes + pos - run->para_offs_bytes;
+	*ppos = pos - run->para_offs_bytes;
 	*bl = run->scblock;
 
 	return 0;
@@ -1624,13 +1568,12 @@ int para_debug_num_runs(Paragraph *para)
 }
 
 int para_debug_run_info(Paragraph *para, int i, size_t *len, SCBlock **scblock,
-                        size_t *scblock_offs, size_t *para_offs)
+                        size_t *para_offs)
 {
 	if ( para->type != PARA_TYPE_TEXT ) return 1;
 	if ( i >= para->n_runs ) return 1;
 
 	*len = para->runs[i].len_bytes;
-	*scblock_offs = para->runs[i].scblock_offs_bytes;
 	*para_offs = para->runs[i].para_offs_bytes;
 	*scblock = para->runs[i].scblock;
 	return 0;
diff --git a/src/frame.h b/src/frame.h
index b60afa1..6145666 100644
--- a/src/frame.h
+++ b/src/frame.h
@@ -137,7 +137,7 @@ extern void show_edit_pos(struct edit_pos a);
 
 extern void add_run(Paragraph *para, SCBlock *scblock,
                     SCBlock *macro_real, SCBlock *contents_top,
-                    size_t offs_bytes, size_t len_bytes,
+                    size_t len_bytes,
                     PangoFontDescription *fdesc, double col[4],
                     int macro_editable);
 
@@ -208,6 +208,6 @@ extern SCBlock *para_scblock(Paragraph *para);
 
 extern int para_debug_num_runs(Paragraph *para);
 extern int para_debug_run_info(Paragraph *para, int i, size_t *len, SCBlock **scblock,
-                               size_t *scblock_offs, size_t *para_offs);
+                               size_t *para_offs);
 
 #endif	/* FRAME_H */
diff --git a/src/sc_editor.c b/src/sc_editor.c
index 5cb1b00..3c5c064 100644
--- a/src/sc_editor.c
+++ b/src/sc_editor.c
@@ -853,7 +853,7 @@ static void insert_text(char *t, SCEditor *e)
 			fprintf(stderr, "Failed to insert paragraph\n");
 			return;
 		}
-		add_run(pnew, ad, NULL, NULL, 0, strlen(t),
+		add_run(pnew, ad, NULL, NULL, strlen(t),
 		        e->cursor_frame->fontdesc, e->cursor_frame->col, 0);
 
 		wrap_frame(e->cursor_frame, e->pc);
@@ -1076,7 +1076,7 @@ static void check_paragraph(struct frame *fr, PangoContext *pc,
 	}
 	scblocks = sc_block_append(scblocks, NULL, NULL, strdup(""), NULL);
 
-	add_run(para, scblocks, NULL, NULL, 0, 0, fr->fontdesc, fr->col, 0);
+	add_run(para, scblocks, NULL, NULL, 0, fr->fontdesc, fr->col, 0);
 	wrap_paragraph(para, pc, fr->w - fr->pad_l - fr->pad_r, 0, 0);
 }
 
diff --git a/src/sc_interp.c b/src/sc_interp.c
index efdffa6..24dd4ee 100644
--- a/src/sc_interp.c
+++ b/src/sc_interp.c
@@ -946,7 +946,7 @@ static int add_text(struct frame *fr, PangoContext *pc, SCBlock *bl,
                     PangoLanguage *lang, int editable, SCInterpreter *scin)
 {
 	const char *text = sc_block_contents(bl);
-	size_t start, len_bytes;
+	size_t len_bytes;
 	PangoFontDescription *fontdesc;
 	double *col;
 	struct sc_state *st = &scin->state[scin->j];
@@ -961,18 +961,10 @@ static int add_text(struct frame *fr, PangoContext *pc, SCBlock *bl,
 	mrb = sc_interp_get_macro_real_block(scin);
 
 	len_bytes = strlen(text);
-	start = 0;
-	do {
-
-		size_t len = strlen(text+start);
-
-		Paragraph *para = last_open_para(fr);
-		add_run(para, bl, mrb, st->macro_contents, start, len,
-		        fontdesc, col, st->macro_editable);
-		set_para_spacing(para, st->paraspace);
-		start += len;
-
-	} while ( start < len_bytes );
+	Paragraph *para = last_open_para(fr);
+	add_run(para, bl, mrb, st->macro_contents, len_bytes,
+	        fontdesc, col, st->macro_editable);
+	set_para_spacing(para, st->paraspace);
 
 	return 0;
 }
@@ -1032,7 +1024,7 @@ static int check_outputs(SCBlock *bl, SCInterpreter *scin)
 		Paragraph *para = last_open_para(fr);
 		struct sc_state *st = &scin->state[scin->j];
 		/* Add a dummy run which we can type into */
-		add_run(para, bl, st->macro_real_block, st->macro_contents, 0, 0,
+		add_run(para, bl, st->macro_real_block, st->macro_contents, 0,
 		        sc_interp_get_fontdesc(scin), fr->col, st->macro_editable);
 		set_newline_at_end(para, bl);
 		close_last_paragraph(fr);