diff options
author | Ingo Schwarze <schwarze@openbsd.org> | 2011-10-16 12:20:34 +0000 |
---|---|---|
committer | Ingo Schwarze <schwarze@openbsd.org> | 2011-10-16 12:20:34 +0000 |
commit | 5a614b48d4e480e6aecbb9f6257de42dee05e3ae (patch) | |
tree | c5f20936909655fb1afd3ac71a4670c9b4d1fb91 /mdoc_macro.c | |
parent | 69a8c4eeef220e21f890c4afe5039fe26bb0249d (diff) | |
download | mandoc-5a614b48d4e480e6aecbb9f6257de42dee05e3ae.tar.gz mandoc-5a614b48d4e480e6aecbb9f6257de42dee05e3ae.tar.zst mandoc-5a614b48d4e480e6aecbb9f6257de42dee05e3ae.zip |
Remove a bunch of useless assignments,
and assert that print_bvspace cannot be called on NULL pointers.
No change in behaviour, none of these were bugs,
but the code becomes easier to understand.
Based on a clang report posted by joerg@; ok kristaps@.
Diffstat (limited to 'mdoc_macro.c')
-rw-r--r-- | mdoc_macro.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/mdoc_macro.c b/mdoc_macro.c index 8b87d4e6..9f8e76be 100644 --- a/mdoc_macro.c +++ b/mdoc_macro.c @@ -1,4 +1,4 @@ -/* $Id: mdoc_macro.c,v 1.111 2011/09/18 14:14:15 schwarze Exp $ */ +/* $Id: mdoc_macro.c,v 1.112 2011/10/16 12:20:34 schwarze Exp $ */ /* * Copyright (c) 2008, 2009, 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv> * Copyright (c) 2010 Ingo Schwarze <schwarze@openbsd.org> @@ -1443,18 +1443,15 @@ blk_part_exp(MACRO_PROT_ARGS) /* Clean-up to leave in a consistent state. */ - if (NULL == head) { + if (NULL == head) if ( ! mdoc_head_alloc(m, line, ppos, tok)) return(0); - head = m->last; - } if (NULL == body) { if ( ! rew_sub(MDOC_HEAD, m, tok, line, ppos)) return(0); if ( ! mdoc_body_alloc(m, line, ppos, tok)) return(0); - body = m->last; } /* Standard appending of delimiters. */ |