[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.51 and 1.65

version 1.51, 2010/11/30 15:24:27 version 1.65, 2011/03/20 16:02:05
Line 1 
Line 1 
 /*      $Id$ */  /*      $Id$ */
 /*  /*
  * Copyright (c) 2008, 2009, 2010 Kristaps Dzonsons <kristaps@bsd.lv>   * Copyright (c) 2008, 2009, 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv>
    * Copyright (c) 2010 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
Line 44  struct man_valid {
Line 45  struct man_valid {
   
 static  int       check_bline(CHKARGS);  static  int       check_bline(CHKARGS);
 static  int       check_eq0(CHKARGS);  static  int       check_eq0(CHKARGS);
   static  int       check_ft(CHKARGS);
 static  int       check_le1(CHKARGS);  static  int       check_le1(CHKARGS);
 static  int       check_ge2(CHKARGS);  static  int       check_ge2(CHKARGS);
 static  int       check_le5(CHKARGS);  static  int       check_le5(CHKARGS);
Line 52  static int   check_part(CHKARGS);
Line 54  static int   check_part(CHKARGS);
 static  int       check_root(CHKARGS);  static  int       check_root(CHKARGS);
 static  int       check_sec(CHKARGS);  static  int       check_sec(CHKARGS);
 static  int       check_text(CHKARGS);  static  int       check_text(CHKARGS);
 static  int       check_title(CHKARGS);  
   
 static  int       post_AT(CHKARGS);  static  int       post_AT(CHKARGS);
 static  int       post_fi(CHKARGS);  static  int       post_fi(CHKARGS);
Line 64  static v_check   posts_at[] = { post_AT, NULL };
Line 65  static v_check   posts_at[] = { post_AT, NULL };
 static  v_check   posts_eq0[] = { check_eq0, NULL };  static  v_check   posts_eq0[] = { check_eq0, NULL };
 static  v_check   posts_fi[] = { check_eq0, post_fi, NULL };  static  v_check   posts_fi[] = { check_eq0, post_fi, NULL };
 static  v_check   posts_le1[] = { check_le1, NULL };  static  v_check   posts_le1[] = { check_le1, NULL };
   static  v_check   posts_ft[] = { check_ft, NULL };
 static  v_check   posts_nf[] = { check_eq0, post_nf, NULL };  static  v_check   posts_nf[] = { check_eq0, post_nf, NULL };
 static  v_check   posts_par[] = { check_par, NULL };  static  v_check   posts_par[] = { check_par, NULL };
 static  v_check   posts_part[] = { check_part, NULL };  static  v_check   posts_part[] = { check_part, NULL };
 static  v_check   posts_sec[] = { check_sec, NULL };  static  v_check   posts_sec[] = { check_sec, NULL };
 static  v_check   posts_th[] = { check_ge2, check_le5, check_title, post_TH, NULL };  static  v_check   posts_th[] = { check_ge2, check_le5, post_TH, NULL };
 static  v_check   posts_uc[] = { post_UC, NULL };  static  v_check   posts_uc[] = { post_UC, NULL };
 static  v_check   posts_vb[] = { check_le1, post_nf, NULL };  
 static  v_check   pres_bline[] = { check_bline, NULL };  static  v_check   pres_bline[] = { check_bline, NULL };
   
   
Line 79  static const struct man_valid man_valids[MAN_MAX] = {
Line 80  static const struct man_valid man_valids[MAN_MAX] = {
         { pres_bline, posts_th }, /* TH */          { pres_bline, posts_th }, /* TH */
         { pres_bline, posts_sec }, /* SH */          { pres_bline, posts_sec }, /* SH */
         { pres_bline, posts_sec }, /* SS */          { pres_bline, posts_sec }, /* SS */
         { pres_bline, posts_par }, /* TP */          { pres_bline, NULL }, /* TP */
         { pres_bline, posts_par }, /* LP */          { pres_bline, posts_par }, /* LP */
         { pres_bline, posts_par }, /* PP */          { pres_bline, posts_par }, /* PP */
         { pres_bline, posts_par }, /* P */          { pres_bline, posts_par }, /* P */
         { pres_bline, posts_par }, /* IP */          { pres_bline, NULL }, /* IP */
         { pres_bline, posts_par }, /* HP */          { pres_bline, NULL }, /* HP */
         { NULL, NULL }, /* SM */          { NULL, NULL }, /* SM */
         { NULL, NULL }, /* SB */          { NULL, NULL }, /* SB */
         { NULL, NULL }, /* BI */          { NULL, NULL }, /* BI */
Line 97  static const struct man_valid man_valids[MAN_MAX] = {
Line 98  static const struct man_valid man_valids[MAN_MAX] = {
         { NULL, NULL }, /* IR */          { NULL, NULL }, /* IR */
         { NULL, NULL }, /* RI */          { NULL, NULL }, /* RI */
         { NULL, posts_eq0 }, /* na */ /* FIXME: should warn only. */          { NULL, posts_eq0 }, /* na */ /* FIXME: should warn only. */
         { NULL, NULL }, /* i */  
         { NULL, posts_le1 }, /* sp */ /* FIXME: should warn only. */          { NULL, posts_le1 }, /* sp */ /* FIXME: should warn only. */
         { pres_bline, posts_nf }, /* nf */          { pres_bline, posts_nf }, /* nf */
         { pres_bline, posts_fi }, /* fi */          { pres_bline, posts_fi }, /* fi */
         { NULL, NULL }, /* r */  
         { NULL, NULL }, /* RE */          { NULL, NULL }, /* RE */
         { NULL, posts_part }, /* RS */          { NULL, posts_part }, /* RS */
         { NULL, NULL }, /* DT */          { NULL, NULL }, /* DT */
         { NULL, posts_uc }, /* UC */          { NULL, posts_uc }, /* UC */
         { NULL, NULL }, /* PD */          { NULL, NULL }, /* PD */
         { NULL, posts_le1 }, /* Sp */ /* FIXME: should warn only. */  
         { pres_bline, posts_vb }, /* Vb */ /* FIXME: should warn only. */  
         { pres_bline, posts_fi }, /* Ve */  
         { NULL, posts_at }, /* AT */          { NULL, posts_at }, /* AT */
         { NULL, NULL }, /* in */          { NULL, NULL }, /* in */
           { NULL, posts_ft }, /* ft */
 };  };
   
   
Line 120  man_valid_pre(struct man *m, struct man_node *n)
Line 117  man_valid_pre(struct man *m, struct man_node *n)
 {  {
         v_check         *cp;          v_check         *cp;
   
         if (MAN_TEXT == n->type)          switch (n->type) {
           case (MAN_TEXT):
                   /* FALLTHROUGH */
           case (MAN_ROOT):
                   /* FALLTHROUGH */
           case (MAN_EQN):
                   /* FALLTHROUGH */
           case (MAN_TBL):
                 return(1);                  return(1);
         if (MAN_ROOT == n->type)          default:
                 return(1);                  break;
           }
   
         if (NULL == (cp = man_valids[n->tok].pres))          if (NULL == (cp = man_valids[n->tok].pres))
                 return(1);                  return(1);
Line 148  man_valid_post(struct man *m)
Line 153  man_valid_post(struct man *m)
                 return(check_text(m, m->last));                  return(check_text(m, m->last));
         case (MAN_ROOT):          case (MAN_ROOT):
                 return(check_root(m, m->last));                  return(check_root(m, m->last));
           case (MAN_EQN):
                   /* FALLTHROUGH */
           case (MAN_TBL):
                   return(1);
         default:          default:
                 break;                  break;
         }          }
Line 167  check_root(CHKARGS) 
Line 176  check_root(CHKARGS) 
 {  {
   
         if (MAN_BLINE & m->flags)          if (MAN_BLINE & m->flags)
                 return(man_nmsg(m, n, MANDOCERR_SCOPEEXIT));                  man_nmsg(m, n, MANDOCERR_SCOPEEXIT);
         if (MAN_ELINE & m->flags)          else if (MAN_ELINE & m->flags)
                 return(man_nmsg(m, n, MANDOCERR_SCOPEEXIT));                  man_nmsg(m, n, MANDOCERR_SCOPEEXIT);
   
         m->flags &= ~MAN_BLINE;          m->flags &= ~MAN_BLINE;
         m->flags &= ~MAN_ELINE;          m->flags &= ~MAN_ELINE;
Line 178  check_root(CHKARGS) 
Line 187  check_root(CHKARGS) 
                 man_nmsg(m, n, MANDOCERR_NODOCBODY);                  man_nmsg(m, n, MANDOCERR_NODOCBODY);
                 return(0);                  return(0);
         } else if (NULL == m->meta.title) {          } else if (NULL == m->meta.title) {
                 if ( ! man_nmsg(m, n, MANDOCERR_NOTITLE))                  man_nmsg(m, n, MANDOCERR_NOTITLE);
                         return(0);  
                 /*                  /*
                  * 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).
                  *  
                  * FIXME: this should be in man_action.c.  
                  */                   */
   
                 m->meta.title = mandoc_strdup("unknown");                  m->meta.title = mandoc_strdup("unknown");
                 m->meta.date = time(NULL);  
                 m->meta.msec = mandoc_strdup("1");                  m->meta.msec = mandoc_strdup("1");
                   m->meta.date = mandoc_normdate
                           (m->parse, NULL, n->line, n->pos);
         }          }
   
         return(1);          return(1);
Line 196  check_root(CHKARGS) 
Line 205  check_root(CHKARGS) 
   
   
 static int  static int
 check_title(CHKARGS)  
 {  
         const char      *p;  
   
         assert(n->child);  
         /* FIXME: is this sufficient? */  
         if ('\0' == *n->child->string) {  
                 man_nmsg(m, n, MANDOCERR_SYNTARGCOUNT);  
                 return(0);  
         }  
   
         for (p = n->child->string; '\0' != *p; p++)  
                 if (isalpha((u_char)*p) && ! isupper((u_char)*p))  
                         if ( ! man_nmsg(m, n, MANDOCERR_UPPERCASE))  
                                 return(0);  
   
         return(1);  
 }  
   
   
 static int  
 check_text(CHKARGS)  check_text(CHKARGS)
 {  {
         char            *p;          char            *p;
Line 235  check_text(CHKARGS) 
Line 223  check_text(CHKARGS) 
                 if ('\t' == *p) {                  if ('\t' == *p) {
                         if (MAN_LITERAL & m->flags)                          if (MAN_LITERAL & m->flags)
                                 continue;                                  continue;
                         if (man_pmsg(m, n->line, pos, MANDOCERR_BADTAB))                          man_pmsg(m, n->line, pos, MANDOCERR_BADTAB);
                                 continue;                          continue;
                         return(0);  
                 }                  }
   
                 /* Check the special character. */                  /* Check the special character. */
Line 260  check_##name(CHKARGS) \
Line 247  check_##name(CHKARGS) \
 { \  { \
         if (n->nchild ineq (x)) \          if (n->nchild ineq (x)) \
                 return(1); \                  return(1); \
         man_vmsg(m, MANDOCERR_SYNTARGCOUNT, n->line, n->pos, \          mandoc_vmsg(MANDOCERR_ARGCOUNT, m->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(0); \          return(1); \
 }  }
   
 INEQ_DEFINE(0, ==, eq0)  INEQ_DEFINE(0, ==, eq0)
Line 271  INEQ_DEFINE(1, <=, le1)
Line 258  INEQ_DEFINE(1, <=, le1)
 INEQ_DEFINE(2, >=, ge2)  INEQ_DEFINE(2, >=, ge2)
 INEQ_DEFINE(5, <=, le5)  INEQ_DEFINE(5, <=, le5)
   
   static int
   check_ft(CHKARGS)
   {
           char    *cp;
           int      ok;
   
           if (0 == n->nchild)
                   return(1);
   
           ok = 0;
           cp = n->child->string;
           switch (*cp) {
           case ('1'):
                   /* FALLTHROUGH */
           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) {
                   mandoc_vmsg
                           (MANDOCERR_BADFONT, m->parse,
                            n->line, n->pos, "%s", cp);
                   *cp = '\0';
           }
   
           if (1 < n->nchild)
                   mandoc_vmsg
                           (MANDOCERR_ARGCOUNT, m->parse, n->line,
                            n->pos, "want one child (have %d)",
                            n->nchild);
   
           return(1);
   }
   
 static int  static int
 check_sec(CHKARGS)  check_sec(CHKARGS)
 {  {
Line 280  check_sec(CHKARGS)
Line 322  check_sec(CHKARGS)
                 man_nmsg(m, n, MANDOCERR_SYNTARGCOUNT);                  man_nmsg(m, n, MANDOCERR_SYNTARGCOUNT);
                 return(0);                  return(0);
         } else if (MAN_BODY == n->type && 0 == n->nchild)          } else if (MAN_BODY == n->type && 0 == n->nchild)
                 return(man_nmsg(m, n, MANDOCERR_NOBODY));                  man_nmsg(m, n, MANDOCERR_NOBODY);
   
         return(1);          return(1);
 }  }
Line 291  check_part(CHKARGS)
Line 333  check_part(CHKARGS)
 {  {
   
         if (MAN_BODY == n->type && 0 == n->nchild)          if (MAN_BODY == n->type && 0 == n->nchild)
                 return(man_nmsg(m, n, MANDOCERR_NOBODY));                  man_nmsg(m, n, MANDOCERR_NOBODY);
   
         return(1);          return(1);
 }  }
   
Line 300  static int
Line 343  static int
 check_par(CHKARGS)  check_par(CHKARGS)
 {  {
   
         if (MAN_BODY == n->type)          switch (n->type) {
                 switch (n->tok) {          case (MAN_BLOCK):
                 case (MAN_IP):                  if (0 == n->body->nchild)
                         /* FALLTHROUGH */                          man_node_delete(m, n);
                 case (MAN_HP):                  break;
                         /* FALLTHROUGH */          case (MAN_BODY):
                 case (MAN_TP):                  if (0 == n->nchild)
                         /* Body-less lists are ok. */                          man_nmsg(m, n, MANDOCERR_IGNPAR);
                         break;                  break;
                 default:          case (MAN_HEAD):
                         if (n->nchild)                  if (n->nchild)
                                 break;                          man_nmsg(m, n, MANDOCERR_ARGSLOST);
                         return(man_nmsg(m, n, MANDOCERR_NOBODY));                  break;
                 }          default:
         if (MAN_HEAD == n->type)                  break;
                 switch (n->tok) {          }
                 case (MAN_PP):  
                         /* FALLTHROUGH */  
                 case (MAN_P):  
                         /* FALLTHROUGH */  
                 case (MAN_LP):  
                         if (0 == n->nchild)  
                                 break;  
                         return(man_nmsg(m, n, MANDOCERR_ARGSLOST));  
                 default:  
                         if (n->nchild)  
                                 break;  
                         return(man_nmsg(m, n, MANDOCERR_NOARGS));  
                 }  
   
         return(1);          return(1);
 }  }
Line 350  check_bline(CHKARGS)
Line 380  check_bline(CHKARGS)
 static int  static int
 post_TH(CHKARGS)  post_TH(CHKARGS)
 {  {
           const char      *p;
           int              line, pos;
   
         if (m->meta.title)          if (m->meta.title)
                 free(m->meta.title);                  free(m->meta.title);
Line 359  post_TH(CHKARGS)
Line 391  post_TH(CHKARGS)
                 free(m->meta.source);                  free(m->meta.source);
         if (m->meta.msec)          if (m->meta.msec)
                 free(m->meta.msec);                  free(m->meta.msec);
         if (m->meta.rawdate)          if (m->meta.date)
                 free(m->meta.rawdate);                  free(m->meta.date);
   
         m->meta.title = m->meta.vol = m->meta.rawdate =          line = n->line;
           pos = n->pos;
           m->meta.title = m->meta.vol = m->meta.date =
                 m->meta.msec = m->meta.source = NULL;                  m->meta.msec = m->meta.source = NULL;
         m->meta.date = 0;  
   
         /* ->TITLE<- MSEC DATE SOURCE VOL */          /* ->TITLE<- MSEC DATE SOURCE VOL */
   
         n = n->child;          n = n->child;
         assert(n);          if (n && n->string) {
         m->meta.title = mandoc_strdup(n->string);                  for (p = n->string; '\0' != *p; p++) {
                           /* Only warn about this once... */
                           if (isalpha((u_char)*p) && ! isupper((u_char)*p)) {
                                   man_nmsg(m, n, MANDOCERR_UPPERCASE);
                                   break;
                           }
                   }
                   m->meta.title = mandoc_strdup(n->string);
           } else
                   m->meta.title = mandoc_strdup("");
   
         /* TITLE ->MSEC<- DATE SOURCE VOL */          /* TITLE ->MSEC<- DATE SOURCE VOL */
   
         n = n->next;          if (n)
         assert(n);                  n = n->next;
         m->meta.msec = mandoc_strdup(n->string);          if (n && n->string)
                   m->meta.msec = mandoc_strdup(n->string);
           else
                   m->meta.msec = mandoc_strdup("");
   
         /* TITLE MSEC ->DATE<- SOURCE VOL */          /* TITLE MSEC ->DATE<- SOURCE VOL */
   
         /*          if (n)
          * Try to parse the date.  If this works, stash the epoch (this                  n = n->next;
          * is optimal because we can reformat it in the canonical form).          if (n)
          * If it doesn't parse, isn't specified at all, or is an empty                  pos = n->pos;
          * string, then use the current date.          m->meta.date = mandoc_normdate
          */                  (m->parse, n ? n->string : NULL, line, pos);
   
         n = n->next;  
         if (n && n->string && *n->string) {  
                 m->meta.date = mandoc_a2time  
                         (MTIME_ISO_8601, n->string);  
                 if (0 == m->meta.date) {  
                         man_nmsg(m, n, MANDOCERR_BADDATE);  
                         m->meta.rawdate = mandoc_strdup(n->string);  
                 }  
         } else  
                 m->meta.date = time(NULL);  
   
         /* TITLE MSEC DATE ->SOURCE<- VOL */          /* TITLE MSEC DATE ->SOURCE<- VOL */
   
         if (n && (n = n->next))          if (n && (n = n->next))
Line 432  post_fi(CHKARGS)
Line 466  post_fi(CHKARGS)
 {  {
   
         if ( ! (MAN_LITERAL & m->flags))          if ( ! (MAN_LITERAL & m->flags))
                 man_nmsg(m, n, MANDOCERR_NOSCOPE);                  man_nmsg(m, n, MANDOCERR_WNOSCOPE);
   
         m->flags &= ~MAN_LITERAL;          m->flags &= ~MAN_LITERAL;
         return(1);          return(1);
Legend:
Removed from v.1.51  
changed lines
  Added in v.1.65
CVSweb