[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.84 and 1.85

version 1.84, 2012/07/29 12:35:42 version 1.85, 2012/11/17 00:26:33
Line 35 
Line 35 
 #include "libman.h"  #include "libman.h"
 #include "libmandoc.h"  #include "libmandoc.h"
   
 #define CHKARGS   struct man *m, struct man_node *n  #define CHKARGS   struct man *man, struct man_node *n
   
 typedef int     (*v_check)(CHKARGS);  typedef int     (*v_check)(CHKARGS);
   
Line 123  static const struct man_valid man_valids[MAN_MAX] = {
Line 123  static const struct man_valid man_valids[MAN_MAX] = {
   
   
 int  int
 man_valid_pre(struct man *m, struct man_node *n)  man_valid_pre(struct man *man, struct man_node *n)
 {  {
         v_check         *cp;          v_check         *cp;
   
Line 143  man_valid_pre(struct man *m, struct man_node *n)
Line 143  man_valid_pre(struct man *m, struct man_node *n)
         if (NULL == (cp = man_valids[n->tok].pres))          if (NULL == (cp = man_valids[n->tok].pres))
                 return(1);                  return(1);
         for ( ; *cp; cp++)          for ( ; *cp; cp++)
                 if ( ! (*cp)(m, n))                  if ( ! (*cp)(man, n))
                         return(0);                          return(0);
         return(1);          return(1);
 }  }
   
   
 int  int
 man_valid_post(struct man *m)  man_valid_post(struct man *man)
 {  {
         v_check         *cp;          v_check         *cp;
   
         if (MAN_VALID & m->last->flags)          if (MAN_VALID & man->last->flags)
                 return(1);                  return(1);
         m->last->flags |= MAN_VALID;          man->last->flags |= MAN_VALID;
   
         switch (m->last->type) {          switch (man->last->type) {
         case (MAN_TEXT):          case (MAN_TEXT):
                 check_text(m, m->last);                  check_text(man, man->last);
                 return(1);                  return(1);
         case (MAN_ROOT):          case (MAN_ROOT):
                 return(check_root(m, m->last));                  return(check_root(man, man->last));
         case (MAN_EQN):          case (MAN_EQN):
                 /* FALLTHROUGH */                  /* FALLTHROUGH */
         case (MAN_TBL):          case (MAN_TBL):
Line 172  man_valid_post(struct man *m)
Line 172  man_valid_post(struct man *m)
                 break;                  break;
         }          }
   
         if (NULL == (cp = man_valids[m->last->tok].posts))          if (NULL == (cp = man_valids[man->last->tok].posts))
                 return(1);                  return(1);
         for ( ; *cp; cp++)          for ( ; *cp; cp++)
                 if ( ! (*cp)(m, m->last))                  if ( ! (*cp)(man, man->last))
                         return(0);                          return(0);
   
         return(1);          return(1);
Line 186  static int
Line 186  static int
 check_root(CHKARGS)  check_root(CHKARGS)
 {  {
   
         if (MAN_BLINE & m->flags)          if (MAN_BLINE & man->flags)
                 man_nmsg(m, n, MANDOCERR_SCOPEEXIT);                  man_nmsg(man, n, MANDOCERR_SCOPEEXIT);
         else if (MAN_ELINE & m->flags)          else if (MAN_ELINE & man->flags)
                 man_nmsg(m, n, MANDOCERR_SCOPEEXIT);                  man_nmsg(man, n, MANDOCERR_SCOPEEXIT);
   
         m->flags &= ~MAN_BLINE;          man->flags &= ~MAN_BLINE;
         m->flags &= ~MAN_ELINE;          man->flags &= ~MAN_ELINE;
   
         if (NULL == m->first->child) {          if (NULL == man->first->child) {
                 man_nmsg(m, n, MANDOCERR_NODOCBODY);                  man_nmsg(man, n, MANDOCERR_NODOCBODY);
                 return(0);                  return(0);
         } else if (NULL == m->meta.title) {          } else if (NULL == man->meta.title) {
                 man_nmsg(m, n, MANDOCERR_NOTITLE);                  man_nmsg(man, n, MANDOCERR_NOTITLE);
   
                 /*                  /*
                  * If a title hasn't been set, do so now (by                   * If a title hasn't been set, do so now (by
                  * implication, date and section also aren't set).                   * implication, date and section also aren't set).
                  */                   */
   
                 m->meta.title = mandoc_strdup("unknown");                  man->meta.title = mandoc_strdup("unknown");
                 m->meta.msec = mandoc_strdup("1");                  man->meta.msec = mandoc_strdup("1");
                 m->meta.date = mandoc_normdate                  man->meta.date = mandoc_normdate
                         (m->parse, NULL, n->line, n->pos);                          (man->parse, NULL, n->line, n->pos);
         }          }
   
         return(1);          return(1);
Line 219  check_text(CHKARGS)
Line 219  check_text(CHKARGS)
 {  {
         char            *cp, *p;          char            *cp, *p;
   
         if (MAN_LITERAL & m->flags)          if (MAN_LITERAL & man->flags)
                 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++)
                 man_pmsg(m, n->line, (int)(p - cp), MANDOCERR_BADTAB);                  man_pmsg(man, n->line, (int)(p - cp), MANDOCERR_BADTAB);
 }  }
   
 #define INEQ_DEFINE(x, ineq, name) \  #define INEQ_DEFINE(x, ineq, name) \
Line 233  check_##name(CHKARGS) \
Line 233  check_##name(CHKARGS) \
 { \  { \
         if (n->nchild ineq (x)) \          if (n->nchild ineq (x)) \
                 return(1); \                  return(1); \
         mandoc_vmsg(MANDOCERR_ARGCOUNT, m->parse, n->line, n->pos, \          mandoc_vmsg(MANDOCERR_ARGCOUNT, man->parse, n->line, n->pos, \
                         "line arguments %s %d (have %d)", \                          "line arguments %s %d (have %d)", \
                         #ineq, (x), n->nchild); \                          #ineq, (x), n->nchild); \
         return(1); \          return(1); \
Line 287  post_ft(CHKARGS)
Line 287  post_ft(CHKARGS)
   
         if (0 == ok) {          if (0 == ok) {
                 mandoc_vmsg                  mandoc_vmsg
                         (MANDOCERR_BADFONT, m->parse,                          (MANDOCERR_BADFONT, man->parse,
                          n->line, n->pos, "%s", cp);                           n->line, n->pos, "%s", cp);
                 *cp = '\0';                  *cp = '\0';
         }          }
   
         if (1 < n->nchild)          if (1 < n->nchild)
                 mandoc_vmsg                  mandoc_vmsg
                         (MANDOCERR_ARGCOUNT, m->parse, n->line,                          (MANDOCERR_ARGCOUNT, man->parse, n->line,
                          n->pos, "want one child (have %d)",                           n->pos, "want one child (have %d)",
                          n->nchild);                           n->nchild);
   
Line 306  pre_sec(CHKARGS)
Line 306  pre_sec(CHKARGS)
 {  {
   
         if (MAN_BLOCK == n->type)          if (MAN_BLOCK == n->type)
                 m->flags &= ~MAN_LITERAL;                  man->flags &= ~MAN_LITERAL;
         return(1);          return(1);
 }  }
   
Line 317  post_sec(CHKARGS)
Line 317  post_sec(CHKARGS)
         if ( ! (MAN_HEAD == n->type && 0 == n->nchild))          if ( ! (MAN_HEAD == n->type && 0 == n->nchild))
                 return(1);                  return(1);
   
         man_nmsg(m, n, MANDOCERR_SYNTARGCOUNT);          man_nmsg(man, n, MANDOCERR_SYNTARGCOUNT);
         return(0);          return(0);
 }  }
   
Line 326  check_part(CHKARGS)
Line 326  check_part(CHKARGS)
 {  {
   
         if (MAN_BODY == n->type && 0 == n->nchild)          if (MAN_BODY == n->type && 0 == n->nchild)
                 mandoc_msg(MANDOCERR_ARGCWARN, m->parse, n->line,                  mandoc_msg(MANDOCERR_ARGCWARN, man->parse, n->line,
                                 n->pos, "want children (have none)");                                  n->pos, "want children (have none)");
   
         return(1);          return(1);
Line 340  check_par(CHKARGS)
Line 340  check_par(CHKARGS)
         switch (n->type) {          switch (n->type) {
         case (MAN_BLOCK):          case (MAN_BLOCK):
                 if (0 == n->body->nchild)                  if (0 == n->body->nchild)
                         man_node_delete(m, n);                          man_node_delete(man, n);
                 break;                  break;
         case (MAN_BODY):          case (MAN_BODY):
                 if (0 == n->nchild)                  if (0 == n->nchild)
                         man_nmsg(m, n, MANDOCERR_IGNPAR);                          man_nmsg(man, n, MANDOCERR_IGNPAR);
                 break;                  break;
         case (MAN_HEAD):          case (MAN_HEAD):
                 if (n->nchild)                  if (n->nchild)
                         man_nmsg(m, n, MANDOCERR_ARGSLOST);                          man_nmsg(man, n, MANDOCERR_ARGSLOST);
                 break;                  break;
         default:          default:
                 break;                  break;
Line 364  post_IP(CHKARGS)
Line 364  post_IP(CHKARGS)
         switch (n->type) {          switch (n->type) {
         case (MAN_BLOCK):          case (MAN_BLOCK):
                 if (0 == n->head->nchild && 0 == n->body->nchild)                  if (0 == n->head->nchild && 0 == n->body->nchild)
                         man_node_delete(m, n);                          man_node_delete(man, n);
                 break;                  break;
         case (MAN_BODY):          case (MAN_BODY):
                 if (0 == n->parent->head->nchild && 0 == n->nchild)                  if (0 == n->parent->head->nchild && 0 == n->nchild)
                         man_nmsg(m, n, MANDOCERR_IGNPAR);                          man_nmsg(man, n, MANDOCERR_IGNPAR);
                 break;                  break;
         default:          default:
                 break;                  break;
Line 382  post_TH(CHKARGS)
Line 382  post_TH(CHKARGS)
         const char      *p;          const char      *p;
         int              line, pos;          int              line, pos;
   
         if (m->meta.title)          free(man->meta.title);
                 free(m->meta.title);          free(man->meta.vol);
         if (m->meta.vol)          free(man->meta.source);
                 free(m->meta.vol);          free(man->meta.msec);
         if (m->meta.source)          free(man->meta.date);
                 free(m->meta.source);  
         if (m->meta.msec)  
                 free(m->meta.msec);  
         if (m->meta.date)  
                 free(m->meta.date);  
   
         line = n->line;          line = n->line;
         pos = n->pos;          pos = n->pos;
         m->meta.title = m->meta.vol = m->meta.date =          man->meta.title = man->meta.vol = man->meta.date =
                 m->meta.msec = m->meta.source = NULL;                  man->meta.msec = man->meta.source = NULL;
   
         /* ->TITLE<- MSEC DATE SOURCE VOL */          /* ->TITLE<- MSEC DATE SOURCE VOL */
   
Line 406  post_TH(CHKARGS)
Line 401  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)) {
                                 man_nmsg(m, n, MANDOCERR_UPPERCASE);                                  man_nmsg(man, n, MANDOCERR_UPPERCASE);
                                 break;                                  break;
                         }                          }
                 }                  }
                 m->meta.title = mandoc_strdup(n->string);                  man->meta.title = mandoc_strdup(n->string);
         } else          } else
                 m->meta.title = mandoc_strdup("");                  man->meta.title = mandoc_strdup("");
   
         /* TITLE ->MSEC<- DATE SOURCE VOL */          /* TITLE ->MSEC<- DATE SOURCE VOL */
   
         if (n)          if (n)
                 n = n->next;                  n = n->next;
         if (n && n->string)          if (n && n->string)
                 m->meta.msec = mandoc_strdup(n->string);                  man->meta.msec = mandoc_strdup(n->string);
         else          else
                 m->meta.msec = mandoc_strdup("");                  man->meta.msec = mandoc_strdup("");
   
         /* TITLE MSEC ->DATE<- SOURCE VOL */          /* TITLE MSEC ->DATE<- SOURCE VOL */
   
Line 429  post_TH(CHKARGS)
Line 424  post_TH(CHKARGS)
                 n = n->next;                  n = n->next;
         if (n && n->string && '\0' != n->string[0]) {          if (n && n->string && '\0' != n->string[0]) {
                 pos = n->pos;                  pos = n->pos;
                 m->meta.date = mandoc_normdate                  man->meta.date = mandoc_normdate
                     (m->parse, n->string, line, pos);                      (man->parse, n->string, line, pos);
         } else          } else
                 m->meta.date = mandoc_strdup("");                  man->meta.date = mandoc_strdup("");
   
         /* TITLE MSEC DATE ->SOURCE<- VOL */          /* TITLE MSEC DATE ->SOURCE<- VOL */
   
         if (n && (n = n->next))          if (n && (n = n->next))
                 m->meta.source = mandoc_strdup(n->string);                  man->meta.source = mandoc_strdup(n->string);
   
         /* TITLE MSEC DATE SOURCE ->VOL<- */          /* TITLE MSEC DATE SOURCE ->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))
                 m->meta.vol = mandoc_strdup(n->string);                  man->meta.vol = mandoc_strdup(n->string);
         else if ('\0' != m->meta.msec[0] &&          else if ('\0' != man->meta.msec[0] &&
             (NULL != (p = mandoc_a2msec(m->meta.msec))))              (NULL != (p = mandoc_a2msec(man->meta.msec))))
                 m->meta.vol = mandoc_strdup(p);                  man->meta.vol = mandoc_strdup(p);
   
         /*          /*
          * 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(m, m->last);          man_node_delete(man, man->last);
         return(1);          return(1);
 }  }
   
Line 460  static int
Line 455  static int
 post_nf(CHKARGS)  post_nf(CHKARGS)
 {  {
   
         if (MAN_LITERAL & m->flags)          if (MAN_LITERAL & man->flags)
                 man_nmsg(m, n, MANDOCERR_SCOPEREP);                  man_nmsg(man, n, MANDOCERR_SCOPEREP);
   
         m->flags |= MAN_LITERAL;          man->flags |= MAN_LITERAL;
         return(1);          return(1);
 }  }
   
Line 471  static int
Line 466  static int
 post_fi(CHKARGS)  post_fi(CHKARGS)
 {  {
   
         if ( ! (MAN_LITERAL & m->flags))          if ( ! (MAN_LITERAL & man->flags))
                 man_nmsg(m, n, MANDOCERR_WNOSCOPE);                  man_nmsg(man, n, MANDOCERR_WNOSCOPE);
   
         m->flags &= ~MAN_LITERAL;          man->flags &= ~MAN_LITERAL;
         return(1);          return(1);
 }  }
   
Line 511  post_UC(CHKARGS)
Line 506  post_UC(CHKARGS)
                         p = bsd_versions[0];                          p = bsd_versions[0];
         }          }
   
         if (m->meta.source)          free(man->meta.source);
                 free(m->meta.source);          man->meta.source = mandoc_strdup(p);
   
         m->meta.source = mandoc_strdup(p);  
         return(1);          return(1);
 }  }
   
Line 551  post_AT(CHKARGS)
Line 544  post_AT(CHKARGS)
                         p = unix_versions[0];                          p = unix_versions[0];
         }          }
   
         if (m->meta.source)          free(man->meta.source);
                 free(m->meta.source);          man->meta.source = mandoc_strdup(p);
   
         m->meta.source = mandoc_strdup(p);  
         return(1);          return(1);
 }  }
   
Line 569  post_vs(CHKARGS)
Line 560  post_vs(CHKARGS)
         case (MAN_SH):          case (MAN_SH):
                 /* FALLTHROUGH */                  /* FALLTHROUGH */
         case (MAN_SS):          case (MAN_SS):
                 man_nmsg(m, n, MANDOCERR_IGNPAR);                  man_nmsg(man, n, MANDOCERR_IGNPAR);
                 /* FALLTHROUGH */                  /* FALLTHROUGH */
         case (MAN_MAX):          case (MAN_MAX):
                 /*                  /*
                  * Don't warn about this because it occurs in pod2man                   * Don't warn about this because it occurs in pod2man
                  * and would cause considerable (unfixable) warnage.                   * and would cause considerable (unfixable) warnage.
                  */                   */
                 man_node_delete(m, n);                  man_node_delete(man, n);
                 break;                  break;
         default:          default:
                 break;                  break;

Legend:
Removed from v.1.84  
changed lines
  Added in v.1.85

CVSweb