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_validate.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_validate.c')
| -rw-r--r-- | mdoc_validate.c | 11 |
1 files changed, 5 insertions, 6 deletions
diff --git a/mdoc_validate.c b/mdoc_validate.c index 5299253b..56936424 100644 --- a/mdoc_validate.c +++ b/mdoc_validate.c @@ -1,4 +1,4 @@ -/* $Id: mdoc_validate.c,v 1.176 2011/09/02 19:40:18 kristaps Exp $ */ +/* $Id: mdoc_validate.c,v 1.177 2011/10/16 12:20:34 schwarze Exp $ */ /* * Copyright (c) 2008, 2009, 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv> * Copyright (c) 2010, 2011 Ingo Schwarze <schwarze@openbsd.org> @@ -545,12 +545,11 @@ check_text(struct mdoc *m, int ln, int pos, char *p) { char *cp; - cp = p; - for (cp = p; NULL != (p = strchr(p, '\t')); p++) { - if (MDOC_LITERAL & m->flags) - continue; + if (MDOC_LITERAL & m->flags) + return; + + for (cp = p; NULL != (p = strchr(p, '\t')); p++) mdoc_pmsg(m, ln, pos + (int)(p - cp), MANDOCERR_BADTAB); - } } static int |