diff options
author | Kristaps Dzonsons <kristaps@bsd.lv> | 2008-11-28 18:15:29 +0000 |
---|---|---|
committer | Kristaps Dzonsons <kristaps@bsd.lv> | 2008-11-28 18:15:29 +0000 |
commit | afac8c1eebed17d57ca78b1d322b71cc1f48af97 (patch) | |
tree | a001734da1ead46951f16a1907f97fc2aef9065a | |
parent | 53836d6f5d78b6d01bab9dc293ffa2649b8a0341 (diff) | |
download | mandoc-afac8c1eebed17d57ca78b1d322b71cc1f48af97.tar.gz |
Cleaned up exit routine.
Added roffhead and rofftail call.
-rw-r--r-- | roff.c | 42 |
1 files changed, 32 insertions, 10 deletions
@@ -357,23 +357,40 @@ const char *const *tokargnames = tokargnamesp; int roff_free(struct rofftree *tree, int flush) { - int error; + int error, t; struct roffnode *n; - error = 0; + if ( ! flush) + goto end; - for (n = tree->last; n->parent; n = n->parent) - if (tokens[n->tok].ctx == 0) { - roff_warn(tree, NULL, "closing explicit scope " - "of `%s'", toknames[n->tok]); - error = 1; - } + error = 1; - if (0 == error && (ROFF_PRELUDE & tree->state)) { + if (ROFF_PRELUDE & tree->state) { roff_warn(tree, NULL, "prelude never finished"); - error = 1; + goto end; + } + + for (n = tree->last; n->parent; n = n->parent) { + if (0 != tokens[n->tok].ctx) + break; + roff_warn(tree, NULL, "closing explicit scope `%s'", + toknames[n->tok]); + goto end; + } + + while (tree->last) { + t = tree->last->tok; + if ( ! (*tokens[t].cb)(t, tree, NULL, ROFF_EXIT)) + goto end; } + if ( ! (*tree->cb.roffhead)(tree->arg)) + goto end; + + error = 0; + +end: + while (tree->last) roffnode_free(tree); @@ -395,6 +412,11 @@ roff_alloc(const struct roffcb *cb, void *args) (void)memcpy(&tree->cb, cb, sizeof(struct roffcb)); + if ( ! (*tree->cb.roffhead)(args)) { + free(tree); + return(NULL); + } + return(tree); } |