[BACK]Return to man_validate.c CVS log [TXT][DIR] Up to [cvsweb.bsd.lv] / mandoc

Diff for /mandoc/man_validate.c between version 1.111 and 1.147

version 1.111, 2015/02/06 08:28:35 version 1.147, 2019/03/11 13:21:11
Line 1 
Line 1 
 /*      $OpenBSD$ */  /*      $Id$ */
 /*  /*
  * Copyright (c) 2008, 2009, 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv>   * Copyright (c) 2008, 2009, 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv>
  * Copyright (c) 2010, 2012, 2013, 2014 Ingo Schwarze <schwarze@openbsd.org>   * Copyright (c) 2010, 2012-2018 Ingo Schwarze <schwarze@openbsd.org>
  *   *
  * Permission to use, copy, modify, and distribute this software for any   * Permission to use, copy, modify, and distribute this software for any
  * purpose with or without fee is hereby granted, provided that the above   * purpose with or without fee is hereby granted, provided that the above
  * copyright notice and this permission notice appear in all copies.   * copyright notice and this permission notice appear in all copies.
  *   *
  * THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES   * THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHORS DISCLAIM ALL WARRANTIES
  * WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF   * WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
  * MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR   * MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHORS BE LIABLE FOR
  * ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES   * ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
  * WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN   * WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
  * ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF   * ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
Line 24 
Line 24 
 #include <errno.h>  #include <errno.h>
 #include <limits.h>  #include <limits.h>
 #include <stdarg.h>  #include <stdarg.h>
   #include <stdio.h>
 #include <stdlib.h>  #include <stdlib.h>
 #include <string.h>  #include <string.h>
 #include <time.h>  #include <time.h>
   
 #include "man.h"  
 #include "mandoc.h"  
 #include "mandoc_aux.h"  #include "mandoc_aux.h"
 #include "libman.h"  #include "mandoc.h"
   #include "roff.h"
   #include "man.h"
 #include "libmandoc.h"  #include "libmandoc.h"
   #include "roff_int.h"
   #include "libman.h"
   
 #define CHKARGS   struct man *man, struct man_node *n  #define CHKARGS   struct roff_man *man, struct roff_node *n
   
 typedef void    (*v_check)(CHKARGS);  typedef void    (*v_check)(CHKARGS);
   
 static  void      check_eq2(CHKARGS);  static __dead void check_abort(CHKARGS);
 static  void      check_le1(CHKARGS);  
 static  void      check_le5(CHKARGS);  
 static  void      check_par(CHKARGS);  static  void      check_par(CHKARGS);
 static  void      check_part(CHKARGS);  static  void      check_part(CHKARGS);
 static  void      check_root(CHKARGS);  static  void      check_root(CHKARGS);
 static  void      check_text(CHKARGS);  static  void      check_text(CHKARGS);
   
 static  void      post_AT(CHKARGS);  static  void      post_AT(CHKARGS);
   static  void      post_EE(CHKARGS);
   static  void      post_EX(CHKARGS);
 static  void      post_IP(CHKARGS);  static  void      post_IP(CHKARGS);
 static  void      post_vs(CHKARGS);  static  void      post_OP(CHKARGS);
 static  void      post_fi(CHKARGS);  static  void      post_SH(CHKARGS);
 static  void      post_ft(CHKARGS);  
 static  void      post_nf(CHKARGS);  
 static  void      post_TH(CHKARGS);  static  void      post_TH(CHKARGS);
 static  void      post_UC(CHKARGS);  static  void      post_UC(CHKARGS);
 static  void      post_UR(CHKARGS);  static  void      post_UR(CHKARGS);
   static  void      post_in(CHKARGS);
   
 static  v_check man_valids[MAN_MAX] = {  static  const v_check man_valids[MAN_MAX - MAN_TH] = {
         post_vs,    /* br */  
         post_TH,    /* TH */          post_TH,    /* TH */
         NULL,       /* SH */          post_SH,    /* SH */
         NULL,       /* SS */          post_SH,    /* SS */
         NULL,       /* TP */          NULL,       /* TP */
         check_par,  /* LP */          NULL,       /* TQ */
           check_abort,/* LP */
         check_par,  /* PP */          check_par,  /* PP */
         check_par,  /* P */          check_abort,/* P */
         post_IP,    /* IP */          post_IP,    /* IP */
         NULL,       /* HP */          NULL,       /* HP */
         NULL,       /* SM */          NULL,       /* SM */
Line 78  static v_check man_valids[MAN_MAX] = {
Line 80  static v_check man_valids[MAN_MAX] = {
         NULL,       /* I */          NULL,       /* I */
         NULL,       /* IR */          NULL,       /* IR */
         NULL,       /* RI */          NULL,       /* RI */
         post_vs,    /* sp */  
         post_nf,    /* nf */  
         post_fi,    /* fi */  
         NULL,       /* RE */          NULL,       /* RE */
         check_part, /* RS */          check_part, /* RS */
         NULL,       /* DT */          NULL,       /* DT */
         post_UC,    /* UC */          post_UC,    /* UC */
         check_le1,  /* PD */          NULL,       /* PD */
         post_AT,    /* AT */          post_AT,    /* AT */
         NULL,       /* in */          post_in,    /* in */
         post_ft,    /* ft */          NULL,       /* SY */
         check_eq2,  /* OP */          NULL,       /* YS */
         post_nf,    /* EX */          post_OP,    /* OP */
         post_fi,    /* EE */          post_EX,    /* EX */
           post_EE,    /* EE */
         post_UR,    /* UR */          post_UR,    /* UR */
         NULL,       /* UE */          NULL,       /* UE */
         NULL,       /* ll */          post_UR,    /* MT */
           NULL,       /* ME */
 };  };
   
   
   /* Validate the subtree rooted at man->last. */
 void  void
 man_valid_post(struct man *man)  man_validate(struct roff_man *man)
 {  {
         struct man_node *n;          struct roff_node *n;
         v_check         *cp;          const v_check    *cp;
   
           /*
            * Translate obsolete macros such that later code
            * does not need to look for them.
            */
   
         n = man->last;          n = man->last;
         if (n->flags & MAN_VALID)          switch (n->tok) {
                 return;          case MAN_LP:
         n->flags |= MAN_VALID;          case MAN_P:
                   n->tok = MAN_PP;
                   break;
           default:
                   break;
           }
   
           /*
            * Iterate over all children, recursing into each one
            * in turn, depth-first.
            */
   
           man->last = man->last->child;
           while (man->last != NULL) {
                   man_validate(man);
                   if (man->last == n)
                           man->last = man->last->child;
                   else
                           man->last = man->last->next;
           }
   
           /* Finally validate the macro itself. */
   
           man->last = n;
           man->next = ROFF_NEXT_SIBLING;
         switch (n->type) {          switch (n->type) {
         case MAN_TEXT:          case ROFFT_TEXT:
                 check_text(man, n);                  check_text(man, n);
                 break;                  break;
         case MAN_ROOT:          case ROFFT_ROOT:
                 check_root(man, n);                  check_root(man, n);
                 break;                  break;
         case MAN_EQN:          case ROFFT_COMMENT:
                 /* FALLTHROUGH */          case ROFFT_EQN:
         case MAN_TBL:          case ROFFT_TBL:
                 break;                  break;
         default:          default:
                 cp = man_valids + n->tok;                  if (n->tok < ROFF_MAX) {
                           roff_validate(man);
                           break;
                   }
                   assert(n->tok >= MAN_TH && n->tok < MAN_MAX);
                   cp = man_valids + (n->tok - MAN_TH);
                 if (*cp)                  if (*cp)
                         (*cp)(man, n);                          (*cp)(man, n);
                   if (man->last == n)
                           n->flags |= NODE_VALID;
                 break;                  break;
         }          }
 }  }
Line 131  man_valid_post(struct man *man)
Line 168  man_valid_post(struct man *man)
 static void  static void
 check_root(CHKARGS)  check_root(CHKARGS)
 {  {
   
         assert((man->flags & (MAN_BLINE | MAN_ELINE)) == 0);          assert((man->flags & (MAN_BLINE | MAN_ELINE)) == 0);
   
         if (NULL == man->first->child)          if (n->last == NULL || n->last->type == ROFFT_COMMENT)
                 mandoc_msg(MANDOCERR_DOC_EMPTY, man->parse,                  mandoc_msg(MANDOCERR_DOC_EMPTY, n->line, n->pos, NULL);
                     n->line, n->pos, NULL);  
         else          else
                 man->meta.hasbody = 1;                  man->meta.hasbody = 1;
   
         if (NULL == man->meta.title) {          if (NULL == man->meta.title) {
                 mandoc_msg(MANDOCERR_TH_NOTITLE, man->parse,                  mandoc_msg(MANDOCERR_TH_NOTITLE, n->line, n->pos, NULL);
                     n->line, n->pos, NULL);  
   
                 /*                  /*
                  * If a title hasn't been set, do so now (by                   * If a title hasn't been set, do so now (by
Line 152  check_root(CHKARGS)
Line 186  check_root(CHKARGS)
                 man->meta.title = mandoc_strdup("");                  man->meta.title = mandoc_strdup("");
                 man->meta.msec = mandoc_strdup("");                  man->meta.msec = mandoc_strdup("");
                 man->meta.date = man->quick ? mandoc_strdup("") :                  man->meta.date = man->quick ? mandoc_strdup("") :
                     mandoc_normdate(man->parse, NULL, n->line, n->pos);                      mandoc_normdate(man, NULL, n->line, n->pos);
         }          }
   
           if (man->meta.os_e &&
               (man->meta.rcsids & (1 << man->meta.os_e)) == 0)
                   mandoc_msg(MANDOCERR_RCS_MISSING, 0, 0,
                       man->meta.os_e == MANDOC_OS_OPENBSD ?
                       "(OpenBSD)" : "(NetBSD)");
 }  }
   
   static __dead void
   check_abort(CHKARGS)
   {
           abort();
   }
   
 static void  static void
 check_text(CHKARGS)  check_text(CHKARGS)
 {  {
         char            *cp, *p;          char            *cp, *p;
   
         if (MAN_LITERAL & man->flags)          if (n->flags & NODE_NOFILL)
                 return;                  return;
   
         cp = n->string;          cp = n->string;
         for (p = cp; NULL != (p = strchr(p, '\t')); p++)          for (p = cp; NULL != (p = strchr(p, '\t')); p++)
                 mandoc_msg(MANDOCERR_FI_TAB, man->parse,                  mandoc_msg(MANDOCERR_FI_TAB,
                     n->line, n->pos + (p - cp), NULL);                      n->line, n->pos + (int)(p - cp), NULL);
 }  }
   
 #define INEQ_DEFINE(x, ineq, name) \  static void
 static void \  post_EE(CHKARGS)
 check_##name(CHKARGS) \  {
 { \          if ((n->flags & NODE_NOFILL) == 0)
         if (n->nchild ineq (x)) \                  mandoc_msg(MANDOCERR_FI_SKIP, n->line, n->pos, "EE");
                 return; \  
         mandoc_vmsg(MANDOCERR_ARGCOUNT, man->parse, n->line, n->pos, \  
             "line arguments %s %d (have %d)", \  
             #ineq, (x), n->nchild); \  
 }  }
   
 INEQ_DEFINE(2, ==, eq2)  static void
 INEQ_DEFINE(1, <=, le1)  post_EX(CHKARGS)
 INEQ_DEFINE(5, <=, le5)  {
           if (n->flags & NODE_NOFILL)
                   mandoc_msg(MANDOCERR_NF_SKIP, n->line, n->pos, "EX");
   }
   
 static void  static void
 post_UR(CHKARGS)  post_OP(CHKARGS)
 {  {
   
         if (MAN_HEAD == n->type && 1 != n->nchild)          if (n->child == NULL)
                 mandoc_vmsg(MANDOCERR_ARGCOUNT, man->parse, n->line,                  mandoc_msg(MANDOCERR_OP_EMPTY, n->line, n->pos, "OP");
                     n->pos, "line arguments eq 1 (have %d)", n->nchild);          else if (n->child->next != NULL && n->child->next->next != NULL) {
         check_part(man, n);                  n = n->child->next->next;
                   mandoc_msg(MANDOCERR_ARG_EXCESS,
                       n->line, n->pos, "OP ... %s", n->string);
           }
 }  }
   
 static void  static void
 post_ft(CHKARGS)  post_SH(CHKARGS)
 {  {
         char    *cp;          struct roff_node        *nc;
         int      ok;  
   
         if (0 == n->nchild)          if (n->type != ROFFT_BODY || (nc = n->child) == NULL)
                 return;                  return;
   
         ok = 0;          if (nc->tok == MAN_PP && nc->body->child != NULL) {
         cp = n->child->string;                  while (nc->body->last != NULL) {
         switch (*cp) {                          man->next = ROFF_NEXT_CHILD;
         case '1':                          roff_node_relink(man, nc->body->last);
                 /* FALLTHROUGH */                          man->last = n;
         case '2':                  }
                 /* FALLTHROUGH */  
         case '3':  
                 /* FALLTHROUGH */  
         case '4':  
                 /* FALLTHROUGH */  
         case 'I':  
                 /* FALLTHROUGH */  
         case 'P':  
                 /* FALLTHROUGH */  
         case 'R':  
                 if ('\0' == cp[1])  
                         ok = 1;  
                 break;  
         case 'B':  
                 if ('\0' == cp[1] || ('I' == cp[1] && '\0' == cp[2]))  
                         ok = 1;  
                 break;  
         case 'C':  
                 if ('W' == cp[1] && '\0' == cp[2])  
                         ok = 1;  
                 break;  
         default:  
                 break;  
         }          }
   
         if (0 == ok) {          if (nc->tok == MAN_PP || nc->tok == ROFF_sp || nc->tok == ROFF_br) {
                 mandoc_vmsg(MANDOCERR_FT_BAD, man->parse,                  mandoc_msg(MANDOCERR_PAR_SKIP, nc->line, nc->pos,
                     n->line, n->pos, "ft %s", cp);                      "%s after %s", roff_name[nc->tok], roff_name[n->tok]);
                 *cp = '\0';                  roff_node_delete(man, nc);
         }          }
   
         if (1 < n->nchild)          /*
                 mandoc_vmsg(MANDOCERR_ARGCOUNT, man->parse, n->line,           * Trailing PP is empty, so it is deleted by check_par().
                     n->pos, "want one child (have %d)", n->nchild);           * Trailing sp is significant.
            */
   
           if ((nc = n->last) != NULL && nc->tok == ROFF_br) {
                   mandoc_msg(MANDOCERR_PAR_SKIP,
                       nc->line, nc->pos, "%s at the end of %s",
                       roff_name[nc->tok], roff_name[n->tok]);
                   roff_node_delete(man, nc);
           }
 }  }
   
 static void  static void
   post_UR(CHKARGS)
   {
           if (n->type == ROFFT_HEAD && n->child == NULL)
                   mandoc_msg(MANDOCERR_UR_NOHEAD, n->line, n->pos,
                       "%s", roff_name[n->tok]);
           check_part(man, n);
   }
   
   static void
 check_part(CHKARGS)  check_part(CHKARGS)
 {  {
   
         if (n->type == MAN_BODY && n->child == NULL)          if (n->type == ROFFT_BODY && n->child == NULL)
                 mandoc_msg(MANDOCERR_BLK_EMPTY, man->parse,                  mandoc_msg(MANDOCERR_BLK_EMPTY, n->line, n->pos,
                     n->line, n->pos, man_macronames[n->tok]);                      "%s", roff_name[n->tok]);
 }  }
   
 static void  static void
Line 260  check_par(CHKARGS)
Line 301  check_par(CHKARGS)
 {  {
   
         switch (n->type) {          switch (n->type) {
         case MAN_BLOCK:          case ROFFT_BLOCK:
                 if (0 == n->body->nchild)                  if (n->body->child == NULL)
                         man_node_delete(man, n);                          roff_node_delete(man, n);
                 break;                  break;
         case MAN_BODY:          case ROFFT_BODY:
                 if (0 == n->nchild)                  if (n->child != NULL &&
                         mandoc_vmsg(MANDOCERR_PAR_SKIP,                      (n->child->tok == ROFF_sp || n->child->tok == ROFF_br)) {
                             man->parse, n->line, n->pos,                          mandoc_msg(MANDOCERR_PAR_SKIP,
                             "%s empty", man_macronames[n->tok]);                              n->child->line, n->child->pos,
                               "%s after %s", roff_name[n->child->tok],
                               roff_name[n->tok]);
                           roff_node_delete(man, n->child);
                   }
                   if (n->child == NULL)
                           mandoc_msg(MANDOCERR_PAR_SKIP, n->line, n->pos,
                               "%s empty", roff_name[n->tok]);
                 break;                  break;
         case MAN_HEAD:          case ROFFT_HEAD:
                 if (n->nchild)                  if (n->child != NULL)
                         mandoc_vmsg(MANDOCERR_ARG_SKIP,                          mandoc_msg(MANDOCERR_ARG_SKIP,
                             man->parse, n->line, n->pos,                              n->line, n->pos, "%s %s%s",
                             "%s %s%s", man_macronames[n->tok],                              roff_name[n->tok], n->child->string,
                             n->child->string,                              n->child->next != NULL ? " ..." : "");
                             n->nchild > 1 ? " ..." : "");  
                 break;                  break;
         default:          default:
                 break;                  break;
Line 288  post_IP(CHKARGS)
Line 335  post_IP(CHKARGS)
 {  {
   
         switch (n->type) {          switch (n->type) {
         case MAN_BLOCK:          case ROFFT_BLOCK:
                 if (0 == n->head->nchild && 0 == n->body->nchild)                  if (n->head->child == NULL && n->body->child == NULL)
                         man_node_delete(man, n);                          roff_node_delete(man, n);
                 break;                  break;
         case MAN_BODY:          case ROFFT_BODY:
                 if (0 == n->parent->head->nchild && 0 == n->nchild)                  if (n->parent->head->child == NULL && n->child == NULL)
                         mandoc_vmsg(MANDOCERR_PAR_SKIP,                          mandoc_msg(MANDOCERR_PAR_SKIP, n->line, n->pos,
                             man->parse, n->line, n->pos,                              "%s empty", roff_name[n->tok]);
                             "%s empty", man_macronames[n->tok]);  
                 break;                  break;
         default:          default:
                 break;                  break;
Line 306  post_IP(CHKARGS)
Line 352  post_IP(CHKARGS)
 static void  static void
 post_TH(CHKARGS)  post_TH(CHKARGS)
 {  {
         struct man_node *nb;          struct roff_node *nb;
         const char      *p;          const char      *p;
   
         check_le5(man, n);  
   
         free(man->meta.title);          free(man->meta.title);
         free(man->meta.vol);          free(man->meta.vol);
         free(man->meta.source);          free(man->meta.os);
         free(man->meta.msec);          free(man->meta.msec);
         free(man->meta.date);          free(man->meta.date);
   
         man->meta.title = man->meta.vol = man->meta.date =          man->meta.title = man->meta.vol = man->meta.date =
             man->meta.msec = man->meta.source = NULL;              man->meta.msec = man->meta.os = NULL;
   
         nb = n;          nb = n;
   
         /* ->TITLE<- MSEC DATE SOURCE VOL */          /* ->TITLE<- MSEC DATE OS VOL */
   
         n = n->child;          n = n->child;
         if (n && n->string) {          if (n && n->string) {
Line 330  post_TH(CHKARGS)
Line 374  post_TH(CHKARGS)
                         /* Only warn about this once... */                          /* Only warn about this once... */
                         if (isalpha((unsigned char)*p) &&                          if (isalpha((unsigned char)*p) &&
                             ! isupper((unsigned char)*p)) {                              ! isupper((unsigned char)*p)) {
                                 mandoc_vmsg(MANDOCERR_TITLE_CASE,                                  mandoc_msg(MANDOCERR_TITLE_CASE, n->line,
                                     man->parse, n->line,                                      n->pos + (int)(p - n->string),
                                     n->pos + (p - n->string),  
                                     "TH %s", n->string);                                      "TH %s", n->string);
                                 break;                                  break;
                         }                          }
Line 340  post_TH(CHKARGS)
Line 383  post_TH(CHKARGS)
                 man->meta.title = mandoc_strdup(n->string);                  man->meta.title = mandoc_strdup(n->string);
         } else {          } else {
                 man->meta.title = mandoc_strdup("");                  man->meta.title = mandoc_strdup("");
                 mandoc_msg(MANDOCERR_TH_NOTITLE, man->parse,                  mandoc_msg(MANDOCERR_TH_NOTITLE, nb->line, nb->pos, "TH");
                     nb->line, nb->pos, "TH");  
         }          }
   
         /* TITLE ->MSEC<- DATE SOURCE VOL */          /* TITLE ->MSEC<- DATE OS VOL */
   
         if (n)          if (n)
                 n = n->next;                  n = n->next;
Line 352  post_TH(CHKARGS)
Line 394  post_TH(CHKARGS)
                 man->meta.msec = mandoc_strdup(n->string);                  man->meta.msec = mandoc_strdup(n->string);
         else {          else {
                 man->meta.msec = mandoc_strdup("");                  man->meta.msec = mandoc_strdup("");
                 mandoc_vmsg(MANDOCERR_MSEC_MISSING, man->parse,                  mandoc_msg(MANDOCERR_MSEC_MISSING,
                     nb->line, nb->pos, "TH %s", man->meta.title);                      nb->line, nb->pos, "TH %s", man->meta.title);
         }          }
   
         /* TITLE MSEC ->DATE<- SOURCE VOL */          /* TITLE MSEC ->DATE<- OS VOL */
   
         if (n)          if (n)
                 n = n->next;                  n = n->next;
         if (n && n->string && '\0' != n->string[0]) {          if (n && n->string && '\0' != n->string[0]) {
                 man->meta.date = man->quick ?                  man->meta.date = man->quick ?
                     mandoc_strdup(n->string) :                      mandoc_strdup(n->string) :
                     mandoc_normdate(man->parse, n->string,                      mandoc_normdate(man, n->string, n->line, n->pos);
                         n->line, n->pos);  
         } else {          } else {
                 man->meta.date = mandoc_strdup("");                  man->meta.date = mandoc_strdup("");
                 mandoc_msg(MANDOCERR_DATE_MISSING, man->parse,                  mandoc_msg(MANDOCERR_DATE_MISSING,
                     n ? n->line : nb->line,                      n ? n->line : nb->line,
                     n ? n->pos : nb->pos, "TH");                      n ? n->pos : nb->pos, "TH");
         }          }
   
         /* TITLE MSEC DATE ->SOURCE<- VOL */          /* TITLE MSEC DATE ->OS<- VOL */
   
         if (n && (n = n->next))          if (n && (n = n->next))
                 man->meta.source = mandoc_strdup(n->string);                  man->meta.os = mandoc_strdup(n->string);
         else if (man->defos != NULL)          else if (man->os_s != NULL)
                 man->meta.source = mandoc_strdup(man->defos);                  man->meta.os = mandoc_strdup(man->os_s);
           if (man->meta.os_e == MANDOC_OS_OTHER && man->meta.os != NULL) {
                   if (strstr(man->meta.os, "OpenBSD") != NULL)
                           man->meta.os_e = MANDOC_OS_OPENBSD;
                   else if (strstr(man->meta.os, "NetBSD") != NULL)
                           man->meta.os_e = MANDOC_OS_NETBSD;
           }
   
         /* TITLE MSEC DATE SOURCE ->VOL<- */          /* TITLE MSEC DATE OS ->VOL<- */
         /* If missing, use the default VOL name for MSEC. */          /* If missing, use the default VOL name for MSEC. */
   
         if (n && (n = n->next))          if (n && (n = n->next))
Line 388  post_TH(CHKARGS)
Line 435  post_TH(CHKARGS)
             (NULL != (p = mandoc_a2msec(man->meta.msec))))              (NULL != (p = mandoc_a2msec(man->meta.msec))))
                 man->meta.vol = mandoc_strdup(p);                  man->meta.vol = mandoc_strdup(p);
   
           if (n != NULL && (n = n->next) != NULL)
                   mandoc_msg(MANDOCERR_ARG_EXCESS,
                       n->line, n->pos, "TH ... %s", n->string);
   
         /*          /*
          * Remove the `TH' node after we've processed it for our           * Remove the `TH' node after we've processed it for our
          * meta-data.           * meta-data.
          */           */
         man_node_delete(man, man->last);          roff_node_delete(man, man->last);
 }  }
   
 static void  static void
 post_nf(CHKARGS)  
 {  
   
         if (man->flags & MAN_LITERAL)  
                 mandoc_msg(MANDOCERR_NF_SKIP, man->parse,  
                     n->line, n->pos, "nf");  
   
         man->flags |= MAN_LITERAL;  
 }  
   
 static void  
 post_fi(CHKARGS)  
 {  
   
         if ( ! (MAN_LITERAL & man->flags))  
                 mandoc_msg(MANDOCERR_FI_SKIP, man->parse,  
                     n->line, n->pos, "fi");  
   
         man->flags &= ~MAN_LITERAL;  
 }  
   
 static void  
 post_UC(CHKARGS)  post_UC(CHKARGS)
 {  {
         static const char * const bsd_versions[] = {          static const char * const bsd_versions[] = {
Line 432  post_UC(CHKARGS)
Line 461  post_UC(CHKARGS)
   
         n = n->child;          n = n->child;
   
         if (NULL == n || MAN_TEXT != n->type)          if (n == NULL || n->type != ROFFT_TEXT)
                 p = bsd_versions[0];                  p = bsd_versions[0];
         else {          else {
                 s = n->string;                  s = n->string;
Line 450  post_UC(CHKARGS)
Line 479  post_UC(CHKARGS)
                         p = bsd_versions[0];                          p = bsd_versions[0];
         }          }
   
         free(man->meta.source);          free(man->meta.os);
         man->meta.source = mandoc_strdup(p);          man->meta.os = mandoc_strdup(p);
 }  }
   
 static void  static void
Line 464  post_AT(CHKARGS)
Line 493  post_AT(CHKARGS)
             "System V Release 2",              "System V Release 2",
         };          };
   
           struct roff_node *nn;
         const char      *p, *s;          const char      *p, *s;
         struct man_node *nn;  
   
         n = n->child;          n = n->child;
   
         if (NULL == n || MAN_TEXT != n->type)          if (n == NULL || n->type != ROFFT_TEXT)
                 p = unix_versions[0];                  p = unix_versions[0];
         else {          else {
                 s = n->string;                  s = n->string;
Line 479  post_AT(CHKARGS)
Line 508  post_AT(CHKARGS)
                         p = unix_versions[1];                          p = unix_versions[1];
                 else if (0 == strcmp(s, "5")) {                  else if (0 == strcmp(s, "5")) {
                         nn = n->next;                          nn = n->next;
                         if (nn && MAN_TEXT == nn->type && nn->string[0])                          if (nn != NULL &&
                               nn->type == ROFFT_TEXT &&
                               nn->string[0] != '\0')
                                 p = unix_versions[3];                                  p = unix_versions[3];
                         else                          else
                                 p = unix_versions[2];                                  p = unix_versions[2];
Line 487  post_AT(CHKARGS)
Line 518  post_AT(CHKARGS)
                         p = unix_versions[0];                          p = unix_versions[0];
         }          }
   
         free(man->meta.source);          free(man->meta.os);
         man->meta.source = mandoc_strdup(p);          man->meta.os = mandoc_strdup(p);
 }  }
   
 static void  static void
 post_vs(CHKARGS)  post_in(CHKARGS)
 {  {
           char    *s;
   
         if (n->tok == MAN_sp)          if (n->parent->tok != MAN_TP ||
                 check_le1(man, n);              n->parent->type != ROFFT_HEAD ||
               n->child == NULL ||
         if (NULL != n->prev)              *n->child->string == '+' ||
               *n->child->string == '-')
                 return;                  return;
           mandoc_asprintf(&s, "+%s", n->child->string);
         switch (n->parent->tok) {          free(n->child->string);
         case MAN_SH:          n->child->string = s;
                 /* FALLTHROUGH */  
         case MAN_SS:  
                 mandoc_vmsg(MANDOCERR_PAR_SKIP, man->parse, n->line, n->pos,  
                     "%s after %s", man_macronames[n->tok],  
                     man_macronames[n->parent->tok]);  
                 /* FALLTHROUGH */  
         case MAN_MAX:  
                 /*  
                  * Don't warn about this because it occurs in pod2man  
                  * and would cause considerable (unfixable) warnage.  
                  */  
                 man_node_delete(man, n);  
                 break;  
         default:  
                 break;  
         }  
 }  }
Legend:
Removed from v.1.111  
changed lines
  Added in v.1.147
CVSweb