]> git.cameronkatri.com Git - mandoc.git/blobdiff - mdoc_action.c
Fixed undocumented `-diag' where NULL list item bodies aren't followed by a vspace.
[mandoc.git] / mdoc_action.c
index f1d5c7e3fb8b2784e29aee99d0813a2befec5824..99b392dba5d79631186e7409cd8b883f133b5381 100644 (file)
@@ -1,4 +1,4 @@
-/*     $Id: mdoc_action.c,v 1.22 2009/07/06 11:14:03 kristaps Exp $ */
+/*     $Id: mdoc_action.c,v 1.30 2009/07/17 10:56:57 kristaps Exp $ */
 /*
  * Copyright (c) 2008, 2009 Kristaps Dzonsons <kristaps@kth.se>
  *
 
 #include "libmdoc.h"
 
-enum   merr {
-       EBADDATE,
-       ENOWIDTH,
-       EBADSEC,
-       ETOOLONG,
-       EMALLOC,
-       EUTSNAME,
-       ENUMFMT
-};
-
 #define        PRE_ARGS  struct mdoc *m, const struct mdoc_node *n
 #define        POST_ARGS struct mdoc *m
 
@@ -42,32 +32,27 @@ struct      actions {
        int     (*post)(POST_ARGS);
 };
 
-static int       perr(struct mdoc *, int, int, enum merr, int);
-static int       concat(struct mdoc *, const struct mdoc_node *, 
-                       char *, size_t);
-
 static int       post_ar(POST_ARGS);
+static int       post_at(POST_ARGS);
 static int       post_bl(POST_ARGS);
 static int       post_bl_head(POST_ARGS);
-static int       post_bl_width(POST_ARGS);
 static int       post_bl_tagwidth(POST_ARGS);
+static int       post_bl_width(POST_ARGS);
 static int       post_dd(POST_ARGS);
 static int       post_display(POST_ARGS);
 static int       post_dt(POST_ARGS);
+static int       post_lb(POST_ARGS);
 static int       post_lk(POST_ARGS);
 static int       post_nm(POST_ARGS);
 static int       post_os(POST_ARGS);
 static int       post_prol(POST_ARGS);
 static int       post_sh(POST_ARGS);
+static int       post_st(POST_ARGS);
 static int       post_std(POST_ARGS);
 
 static int       pre_bd(PRE_ARGS);
 static int       pre_dl(PRE_ARGS);
 
-#define        vwarn(m, t) perr((m), (m)->last->line, (m)->last->pos, (t), 0)
-#define        verr(m, t) perr((m), (m)->last->line, (m)->last->pos, (t), 1)
-#define        nerr(m, n, t) perr((m), (n)->line, (n)->pos, (t), 1)
-
 const  struct actions mdoc_actions[MDOC_MAX] = {
        { NULL, NULL }, /* Ap */
        { NULL, post_dd }, /* Dd */ 
@@ -106,7 +91,7 @@ const        struct actions mdoc_actions[MDOC_MAX] = {
        { NULL, NULL }, /* Ot */
        { NULL, NULL }, /* Pa */
        { NULL, post_std }, /* Rv */
-       { NULL, NULL }, /* St */
+       { NULL, post_st }, /* St */
        { NULL, NULL }, /* Va */
        { NULL, NULL }, /* Vt */ 
        { NULL, NULL }, /* Xr */
@@ -124,7 +109,7 @@ const       struct actions mdoc_actions[MDOC_MAX] = {
        { NULL, NULL }, /* Ac */
        { NULL, NULL }, /* Ao */
        { NULL, NULL }, /* Aq */
-       { NULL, NULL }, /* At */ 
+       { NULL, post_at }, /* At */ 
        { NULL, NULL }, /* Bc */
        { NULL, NULL }, /* Bf */ 
        { NULL, NULL }, /* Bo */
@@ -175,7 +160,7 @@ const       struct actions mdoc_actions[MDOC_MAX] = {
        { NULL, NULL }, /* Hf */
        { NULL, NULL }, /* Fr */
        { NULL, NULL }, /* Ud */
-       { NULL, NULL }, /* Lb */
+       { NULL, post_lb }, /* Lb */
        { NULL, NULL }, /* Lp */
        { NULL, post_lk }, /* Lk */
        { NULL, NULL }, /* Mt */
@@ -187,11 +172,15 @@ const     struct actions mdoc_actions[MDOC_MAX] = {
        { NULL, NULL }, /* En */
        { NULL, NULL }, /* Dx */
        { NULL, NULL }, /* %Q */
+       { NULL, NULL }, /* br */
+       { NULL, NULL }, /* sp */
 };
 
+static int       concat(struct mdoc *, const struct mdoc_node *, 
+                       char *, size_t);
 
 #ifdef __linux__
-extern size_t  strlcat(char *, const char *, size_t);
+extern size_t    strlcat(char *, const char *, size_t);
 #endif
 
 
@@ -245,56 +234,17 @@ concat(struct mdoc *m, const struct mdoc_node *n,
        for ( ; n; n = n->next) {
                assert(MDOC_TEXT == n->type);
                if (strlcat(buf, n->string, sz) >= sz)
-                       return(nerr(m, n, ETOOLONG));
+                       return(mdoc_nerr(m, n, ETOOLONG));
                if (NULL == n->next)
                        continue;
                if (strlcat(buf, " ", sz) >= sz)
-                       return(nerr(m, n, ETOOLONG));
+                       return(mdoc_nerr(m, n, ETOOLONG));
        }
 
        return(1);
 }
 
 
-static int
-perr(struct mdoc *m, int line, int pos, enum merr type, int iserr)
-{
-       char            *p;
-
-       p = NULL;
-
-       switch (type) {
-       case (ENUMFMT):
-               p = "bad number format";
-               break;
-       case (ETOOLONG):
-               p = "argument text too long";
-               break;
-       case (EUTSNAME):
-               p = "utsname";
-               break;
-       case (EMALLOC):
-               p = "memory exhausted";
-               break;
-       case (EBADSEC):
-               p = "inappropriate document section in manual section";
-               break;
-       case (ENOWIDTH):
-               p = "cannot determine default width";
-               break;
-       case (EBADDATE):
-               p = "malformed date syntax";
-               break;
-       }
-
-       assert(p);
-       if (iserr)
-               return(mdoc_perr(m, line, pos, p));
-
-       return(mdoc_pwarn(m, line, pos, p));
-}
-
-
 static int
 post_std(POST_ARGS)
 {
@@ -313,12 +263,12 @@ post_std(POST_ARGS)
 
        m->last->args->argv[0].value = calloc(1, sizeof(char *));
        if (NULL == m->last->args->argv[0].value)
-               return(verr(m, EMALLOC));
+               return(mdoc_nerr(m, m->last, EMALLOC));
 
        m->last->args->argv[0].sz = 1;
        m->last->args->argv[0].value[0] = strdup(m->meta.name);
        if (NULL == m->last->args->argv[0].value[0])
-               return(verr(m, EMALLOC));
+               return(mdoc_nerr(m, m->last, EMALLOC));
 
        return(1);
 }
@@ -337,12 +287,87 @@ post_nm(POST_ARGS)
                return(0);
 
        if (NULL == (m->meta.name = strdup(buf)))
-               return(verr(m, EMALLOC));
+               return(mdoc_nerr(m, m->last, EMALLOC));
 
        return(1);
 }
 
 
+static int
+post_lb(POST_ARGS)
+{
+       const char      *p;
+       char            *buf;
+       size_t           sz;
+
+       assert(MDOC_TEXT == m->last->child->type);
+       p = mdoc_a2lib(m->last->child->string);
+       if (NULL == p) {
+               sz = strlen(m->last->child->string) +
+                       2 + strlen("\\(lqlibrary\\(rq");
+               buf = malloc(sz);
+               if (NULL == buf)
+                       return(mdoc_nerr(m, m->last, EMALLOC));
+               (void)snprintf(buf, sz, "library \\(lq%s\\(rq", 
+                               m->last->child->string);
+               free(m->last->child->string);
+               m->last->child->string = buf;
+               return(1);
+       }
+
+       free(m->last->child->string);
+       m->last->child->string = strdup(p);
+       if (NULL == m->last->child->string)
+               return(mdoc_nerr(m, m->last, EMALLOC));
+       return(1);
+}
+
+
+static int
+post_st(POST_ARGS)
+{
+       const char      *p;
+
+       assert(MDOC_TEXT == m->last->child->type);
+       p = mdoc_a2st(m->last->child->string);
+       assert(p);
+       free(m->last->child->string);
+       m->last->child->string = strdup(p);
+       if (NULL == m->last->child->string)
+               return(mdoc_nerr(m, m->last, EMALLOC));
+       return(1);
+}
+
+
+static int
+post_at(POST_ARGS)
+{
+       struct mdoc_node *n;
+       const char       *p;
+
+       if (m->last->child) {
+               assert(MDOC_TEXT == m->last->child->type);
+               p = mdoc_a2att(m->last->child->string);
+               assert(p);
+               free(m->last->child->string);
+               m->last->child->string = strdup(p);
+               if (NULL == m->last->child->string)
+                       return(mdoc_nerr(m, m->last, EMALLOC));
+               return(1);
+       }
+
+       n = m->last;
+       m->next = MDOC_NEXT_CHILD;
+
+       if ( ! mdoc_word_alloc(m, n->line, n->pos, "AT&T UNIX"))
+               return(0);
+
+       m->last = n;
+       m->next = MDOC_NEXT_SIBLING;
+       return(1);
+}
+
+
 static int
 post_sh(POST_ARGS)
 {
@@ -376,7 +401,7 @@ post_sh(POST_ARGS)
                case (9):
                        break;
                default:
-                       return(vwarn(m, EBADSEC));
+                       return(mdoc_nwarn(m, m->last, EBADSEC));
                }
                break;
        default:
@@ -410,9 +435,9 @@ post_dt(POST_ARGS)
 
        if (NULL == (n = m->last->child)) {
                if (NULL == (m->meta.title = strdup("unknown")))
-                       return(verr(m, EMALLOC));
+                       return(mdoc_nerr(m, m->last, EMALLOC));
                if (NULL == (m->meta.vol = strdup("local")))
-                       return(verr(m, EMALLOC));
+                       return(mdoc_nerr(m, m->last, EMALLOC));
                return(post_prol(m));
        }
 
@@ -421,11 +446,11 @@ post_dt(POST_ARGS)
         */
 
        if (NULL == (m->meta.title = strdup(n->string)))
-               return(verr(m, EMALLOC));
+               return(mdoc_nerr(m, m->last, EMALLOC));
 
        if (NULL == (n = n->next)) {
                if (NULL == (m->meta.vol = strdup("local")))
-                       return(verr(m, EMALLOC));
+                       return(mdoc_nerr(m, m->last, EMALLOC));
                return(post_prol(m));
        }
 
@@ -439,13 +464,13 @@ post_dt(POST_ARGS)
        cp = mdoc_a2msec(n->string);
        if (cp) {
                if (NULL == (m->meta.vol = strdup(cp)))
-                       return(verr(m, EMALLOC));
+                       return(mdoc_nerr(m, m->last, EMALLOC));
                errno = 0;
                lval = strtol(n->string, &ep, 10);
                if (n->string[0] != '\0' && *ep == '\0')
                        m->meta.msec = (int)lval;
        } else if (NULL == (m->meta.vol = strdup(n->string)))
-               return(verr(m, EMALLOC));
+               return(mdoc_nerr(m, m->last, EMALLOC));
 
        if (NULL == (n = n->next))
                return(post_prol(m));
@@ -461,16 +486,16 @@ post_dt(POST_ARGS)
        if (cp) {
                free(m->meta.vol);
                if (NULL == (m->meta.vol = strdup(cp)))
-                       return(verr(m, EMALLOC));
+                       return(mdoc_nerr(m, m->last, EMALLOC));
                n = n->next;
        } else {
                cp = mdoc_a2arch(n->string);
                if (NULL == cp) {
                        free(m->meta.vol);
                        if (NULL == (m->meta.vol = strdup(n->string)))
-                               return(verr(m, EMALLOC));
+                               return(mdoc_nerr(m, m->last, EMALLOC));
                } else if (NULL == (m->meta.arch = strdup(cp)))
-                       return(verr(m, EMALLOC));
+                       return(mdoc_nerr(m, m->last, EMALLOC));
        }       
 
        /* Ignore any subsequent parameters... */
@@ -494,17 +519,17 @@ post_os(POST_ARGS)
 
        if (0 == buf[0]) {
                if (-1 == uname(&utsname))
-                       return(verr(m, EUTSNAME));
+                       return(mdoc_nerr(m, m->last, EUTSNAME));
                if (strlcat(buf, utsname.sysname, 64) >= 64)
-                       return(verr(m, ETOOLONG));
+                       return(mdoc_nerr(m, m->last, ETOOLONG));
                if (strlcat(buf, " ", 64) >= 64)
-                       return(verr(m, ETOOLONG));
+                       return(mdoc_nerr(m, m->last, ETOOLONG));
                if (strlcat(buf, utsname.release, 64) >= 64)
-                       return(verr(m, ETOOLONG));
+                       return(mdoc_nerr(m, m->last, ETOOLONG));
        }
 
        if (NULL == (m->meta.os = strdup(buf)))
-               return(verr(m, EMALLOC));
+               return(mdoc_nerr(m, m->last, EMALLOC));
 
        m->flags |= MDOC_PBODY;
        return(post_prol(m));
@@ -539,14 +564,14 @@ post_bl_tagwidth(struct mdoc *m)
        if (n) {
                if (MDOC_TEXT != n->type) {
                        if (0 == (sz = (int)mdoc_macro2len(n->tok)))
-                               if ( ! vwarn(m, ENOWIDTH))
+                               if ( ! mdoc_nwarn(m, m->last, ENOWIDTH))
                                        return(0);
                } else
                        sz = (int)strlen(n->string) + 1;
        } 
 
        if (-1 == snprintf(buf, sizeof(buf), "%dn", sz))
-               return(verr(m, ENUMFMT));
+               return(mdoc_nerr(m, m->last, ENUMFMT));
 
        /*
         * We have to dynamically add this to the macro's argument list.
@@ -561,7 +586,7 @@ post_bl_tagwidth(struct mdoc *m)
                        n->args->argc * sizeof(struct mdoc_argv));
 
        if (NULL == n->args->argv)
-               return(verr(m, EMALLOC));
+               return(mdoc_nerr(m, m->last, EMALLOC));
 
        n->args->argv[sz].arg = MDOC_Width;
        n->args->argv[sz].line = m->last->line;
@@ -570,9 +595,9 @@ post_bl_tagwidth(struct mdoc *m)
        n->args->argv[sz].value = calloc(1, sizeof(char *));
 
        if (NULL == n->args->argv[sz].value)
-               return(verr(m, EMALLOC));
+               return(mdoc_nerr(m, m->last, EMALLOC));
        if (NULL == (n->args->argv[sz].value[0] = strdup(buf)))
-               return(verr(m, EMALLOC));
+               return(mdoc_nerr(m, m->last, EMALLOC));
 
        return(1);
 }
@@ -607,17 +632,17 @@ post_bl_width(struct mdoc *m)
        else if (MDOC_MAX == (tok = mdoc_hash_find(m->htab, p)))
                return(1);
        else if (0 == (width = mdoc_macro2len(tok))) 
-               return(vwarn(m, ENOWIDTH));
+               return(mdoc_nwarn(m, m->last, ENOWIDTH));
 
        /* The value already exists: free and reallocate it. */
 
        if (-1 == snprintf(buf, sizeof(buf), "%zun", width))
-               return(verr(m, ENUMFMT));
+               return(mdoc_nerr(m, m->last, ENUMFMT));
 
        free(m->last->args->argv[i].value[0]);
        m->last->args->argv[i].value[0] = strdup(buf);
        if (NULL == m->last->args->argv[i].value[0])
-               return(verr(m, EMALLOC));
+               return(mdoc_nerr(m, m->last, EMALLOC));
 
        return(1);
 }
@@ -719,9 +744,9 @@ post_lk(POST_ARGS)
        
        n = m->last;
        m->next = MDOC_NEXT_CHILD;
-       /* FIXME: this isn't documented anywhere! */
-       if ( ! mdoc_word_alloc(m, m->last->line,
-                               m->last->pos, "~"))
+
+       /* XXX: this isn't documented anywhere! */
+       if ( ! mdoc_word_alloc(m, m->last->line, m->last->pos, "~"))
                return(0);
 
        m->last = n;
@@ -740,12 +765,10 @@ post_ar(POST_ARGS)
        
        n = m->last;
        m->next = MDOC_NEXT_CHILD;
-       if ( ! mdoc_word_alloc(m, m->last->line,
-                               m->last->pos, "file"))
+       if ( ! mdoc_word_alloc(m, m->last->line, m->last->pos, "file"))
                return(0);
        m->next = MDOC_NEXT_SIBLING;
-       if ( ! mdoc_word_alloc(m, m->last->line, 
-                               m->last->pos, "..."))
+       if ( ! mdoc_word_alloc(m, m->last->line, m->last->pos, "..."))
                return(0);
 
        m->last = n;
@@ -764,7 +787,7 @@ post_dd(POST_ARGS)
                return(0);
 
        if (0 == (m->meta.date = mdoc_atotime(buf))) {
-               if ( ! vwarn(m, EBADDATE))
+               if ( ! mdoc_nwarn(m, m->last, EBADDATE))
                        return(0);
                m->meta.date = time(NULL);
        }