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 /term_ps.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 'term_ps.c')
| -rw-r--r-- | term_ps.c | 7 |
1 files changed, 3 insertions, 4 deletions
@@ -1,4 +1,4 @@ -/* $Id: term_ps.c,v 1.53 2011/09/18 14:14:15 schwarze Exp $ */ +/* $Id: term_ps.c,v 1.54 2011/10/16 12:20:34 schwarze Exp $ */ /* * Copyright (c) 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv> * @@ -488,8 +488,7 @@ pspdf_alloc(char *outopts) pagey = 356; } else if (2 != sscanf(pp, "%ux%u", &pagex, &pagey)) fprintf(stderr, "%s: Unknown paper\n", pp); - } else if (NULL == pp) - pp = "letter"; + } /* * This MUST be defined before any PNT2AFM or AFM2PNT @@ -576,7 +575,7 @@ ps_printf(struct termp *p, const char *fmt, ...) ps_growbuf(p, PS_BUFSLOP); pos = (int)p->ps->psmargcur; - len = vsnprintf(&p->ps->psmarg[pos], PS_BUFSLOP, fmt, ap); + vsnprintf(&p->ps->psmarg[pos], PS_BUFSLOP, fmt, ap); va_end(ap); |