* mdocml: simplify by getting rid of ROFF_ERR in tbl(7) parsing; no
@ 2015-01-14 22:45 schwarze
0 siblings, 0 replies; only message in thread
From: schwarze @ 2015-01-14 22:45 UTC (permalink / raw)
To: source
Log Message:
-----------
simplify by getting rid of ROFF_ERR in tbl(7) parsing; no functional change
Modified Files:
--------------
mdocml:
libroff.h
tbl.c
tbl_layout.c
tbl_opts.c
Revision Data
-------------
Index: tbl_layout.c
===================================================================
RCS file: /home/cvs/mdocml/mdocml/tbl_layout.c,v
retrieving revision 1.30
retrieving revision 1.31
diff -Ltbl_layout.c -Ltbl_layout.c -u -p -r1.30 -r1.31
--- tbl_layout.c
+++ tbl_layout.c
@@ -299,7 +299,7 @@ cell(struct tbl_node *tbl, struct tbl_ro
return(mods(tbl, cell_alloc(tbl, rp, c, vert), ln, p, pos));
}
-int
+void
tbl_layout(struct tbl_node *tbl, int ln, const char *p)
{
struct tbl_row *rp;
@@ -320,18 +320,18 @@ tbl_layout(struct tbl_node *tbl, int ln,
rp = NULL;
continue;
case '\0': /* Next row on next input line. */
- return(1);
+ return;
case '.': /* End of layout. */
pos++;
tbl->part = TBL_PART_DATA;
if (tbl->first_row != NULL)
- return(1);
+ return;
mandoc_msg(MANDOCERR_TBLNOLAYOUT,
tbl->parse, ln, pos, NULL);
rp = mandoc_calloc(1, sizeof(*rp));
cell_alloc(tbl, rp, TBL_CELL_LEFT, 0);
tbl->first_row = tbl->last_row = rp;
- return(1);
+ return;
default: /* Cell. */
break;
}
@@ -345,7 +345,7 @@ tbl_layout(struct tbl_node *tbl, int ln,
tbl->last_row = rp;
}
if ( ! cell(tbl, rp, ln, p, &pos))
- return(1);
+ return;
}
}
Index: tbl_opts.c
===================================================================
RCS file: /home/cvs/mdocml/mdocml/tbl_opts.c,v
retrieving revision 1.15
retrieving revision 1.16
diff -Ltbl_opts.c -Ltbl_opts.c -u -p -r1.15 -r1.16
--- tbl_opts.c
+++ tbl_opts.c
@@ -252,7 +252,7 @@ again: /*
/* NOTREACHED */
}
-int
+void
tbl_option(struct tbl_node *tbl, int ln, const char *p)
{
int pos;
@@ -265,7 +265,4 @@ tbl_option(struct tbl_node *tbl, int ln,
pos = 0;
opt(tbl, ln, p, &pos);
-
- /* Always succeed. */
- return(1);
}
Index: tbl.c
===================================================================
RCS file: /home/cvs/mdocml/mdocml/tbl.c,v
retrieving revision 1.30
retrieving revision 1.31
diff -Ltbl.c -Ltbl.c -u -p -r1.30 -r1.31
--- tbl.c
+++ tbl.c
@@ -55,9 +55,11 @@ tbl_read(struct tbl_node *tbl, int ln, c
switch (tbl->part) {
case TBL_PART_OPTS:
- return(tbl_option(tbl, ln, p) ? ROFF_IGN : ROFF_ERR);
+ tbl_option(tbl, ln, p);
+ return(ROFF_IGN);
case TBL_PART_LAYOUT:
- return(tbl_layout(tbl, ln, p) ? ROFF_IGN : ROFF_ERR);
+ tbl_layout(tbl, ln, p);
+ return(ROFF_IGN);
case TBL_PART_CDATA:
return(tbl_cdata(tbl, ln, p) ? ROFF_TBL : ROFF_IGN);
default:
Index: libroff.h
===================================================================
RCS file: /home/cvs/mdocml/mdocml/libroff.h,v
retrieving revision 1.33
retrieving revision 1.34
diff -Llibroff.h -Llibroff.h -u -p -r1.33 -r1.34
--- libroff.h
+++ libroff.h
@@ -69,8 +69,8 @@ void tbl_restart(int, int, struct tbl_
void tbl_free(struct tbl_node *);
void tbl_reset(struct tbl_node *);
enum rofferr tbl_read(struct tbl_node *, int, const char *, int);
-int tbl_option(struct tbl_node *, int, const char *);
-int tbl_layout(struct tbl_node *, int, const char *);
+void tbl_option(struct tbl_node *, int, const char *);
+void tbl_layout(struct tbl_node *, int, const char *);
int tbl_data(struct tbl_node *, int, const char *);
int tbl_cdata(struct tbl_node *, int, const char *);
const struct tbl_span *tbl_span(struct tbl_node *);
--
To unsubscribe send an email to source+unsubscribe@mdocml.bsd.lv
^ permalink raw reply [flat|nested] only message in thread
only message in thread, other threads:[~2015-01-14 22:45 UTC | newest]
Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2015-01-14 22:45 mdocml: simplify by getting rid of ROFF_ERR in tbl(7) parsing; no schwarze
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).