diff options
| author |   Kristaps Dzonsons <kristaps@bsd.lv> | 2011-03-17 11:09:36 +0000 |
|---|---|---|
| committer | Kristaps Dzonsons <kristaps@bsd.lv> | 2011-03-17 11:09:36 +0000 |
| commit | 8ccf0c87185bc55b87c3903be984e8afa6e46f7c (patch) | |
| tree | 9c58202baf25f26d4404fe09fb6c1e072f71bb6d /mdoc_argv.c | |
| parent | c4ce972539ee830b52aaadfe08c34216692bffd2 (diff) | |
| download | mandoc-8ccf0c87185bc55b87c3903be984e8afa6e46f7c.tar.gz mandoc-8ccf0c87185bc55b87c3903be984e8afa6e46f7c.tar.zst mandoc-8ccf0c87185bc55b87c3903be984e8afa6e46f7c.zip | |
Clean-up in mdoc_argv.c args_checkpunct().
Diffstat (limited to 'mdoc_argv.c')
| -rw-r--r-- | mdoc_argv.c | 50 |
1 files changed, 22 insertions, 28 deletions
diff --git a/mdoc_argv.c b/mdoc_argv.c index 5acf6f48..05d699a7 100644 --- a/mdoc_argv.c +++ b/mdoc_argv.c @@ -1,4 +1,4 @@ -/* $Id: mdoc_argv.c,v 1.68 2011/03/17 10:59:27 kristaps Exp $ */ +/* $Id: mdoc_argv.c,v 1.69 2011/03/17 11:09:36 kristaps Exp $ */ /* * Copyright (c) 2008, 2009, 2010 Kristaps Dzonsons <kristaps@bsd.lv> * @@ -35,7 +35,8 @@ static enum mdocargt argv_a2arg(enum mdoct, const char *); static enum margserr args(struct mdoc *, int, int *, char *, int, char **); -static int args_checkpunct(const char *); +static int args_checkpunct(struct mdoc *, + const char *, int, int, int); static int argv(struct mdoc *, int, struct mdoc_argv *, int *, char *); static int argv_single(struct mdoc *, int, @@ -361,7 +362,6 @@ static enum margserr args(struct mdoc *m, int line, int *pos, char *buf, int fl, char **v) { - int i; char *p, *pp; enum margserr rc; @@ -399,15 +399,8 @@ args(struct mdoc *m, int line, int *pos, *v = &buf[*pos]; - if (ARGS_DELIM & fl && args_checkpunct(&buf[*pos])) { - i = strlen(&buf[*pos]) + *pos; - if (i && ' ' != buf[i - 1]) - return(ARGS_PUNCT); - if (ARGS_NOWARN & fl) - return(ARGS_PUNCT); - mdoc_pmsg(m, line, *pos, MANDOCERR_EOLNSPACE); + if (ARGS_DELIM & fl && args_checkpunct(m, buf, *pos, line, fl)) return(ARGS_PUNCT); - } /* * First handle TABSEP items, restricted to `Bl -column'. This @@ -549,49 +542,50 @@ args(struct mdoc *m, int line, int *pos, * whitespace may separate these tokens. */ static int -args_checkpunct(const char *p) +args_checkpunct(struct mdoc *m, const char *buf, int i, int ln, int fl) { - int i, j; - char buf[DELIMSZ]; + int j; + char dbuf[DELIMSZ]; enum mdelim d; - i = 0; - /* First token must be a close-delimiter. */ - for (j = 0; p[i] && ' ' != p[i] && j < DELIMSZ; j++, i++) - buf[j] = p[i]; + for (j = 0; buf[i] && ' ' != buf[i] && j < DELIMSZ; j++, i++) + dbuf[j] = buf[i]; if (DELIMSZ == j) return(0); - buf[j] = '\0'; - if (DELIM_CLOSE != mandoc_isdelim(buf)) + dbuf[j] = '\0'; + if (DELIM_CLOSE != mandoc_isdelim(dbuf)) return(0); - while (' ' == p[i]) + while (' ' == buf[i]) i++; /* Remaining must NOT be open/none. */ - while (p[i]) { + while (buf[i]) { j = 0; - while (p[i] && ' ' != p[i] && j < DELIMSZ) - buf[j++] = p[i++]; + while (buf[i] && ' ' != buf[i] && j < DELIMSZ) + dbuf[j++] = buf[i++]; if (DELIMSZ == j) return(0); - buf[j] = '\0'; - d = mandoc_isdelim(buf); + dbuf[j] = '\0'; + d = mandoc_isdelim(dbuf); if (DELIM_NONE == d || DELIM_OPEN == d) return(0); - while (' ' == p[i]) + while (' ' == buf[i]) i++; } - return('\0' == p[i]); + if ( ! (ARGS_NOWARN & fl) && i && ' ' == buf[i - 1]) + mdoc_pmsg(m, ln, i - 1, MANDOCERR_EOLNSPACE); + + return('\0' == buf[i]); } /* |