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

Diff for /mandoc/mdoc_validate.c between version 1.87 and 1.91

version 1.87, 2010/05/30 22:56:02 version 1.91, 2010/06/03 13:44:36
Line 34 
Line 34 
 /* FIXME: .Bl -diag can't have non-text children in HEAD. */  /* FIXME: .Bl -diag can't have non-text children in HEAD. */
 /* TODO: ignoring Pp (it's superfluous in some invocations). */  /* TODO: ignoring Pp (it's superfluous in some invocations). */
   
 #define PRE_ARGS  struct mdoc *mdoc, const struct mdoc_node *n  #define PRE_ARGS  struct mdoc *mdoc, struct mdoc_node *n
 #define POST_ARGS struct mdoc *mdoc  #define POST_ARGS struct mdoc *mdoc
   
 typedef int     (*v_pre)(PRE_ARGS);  typedef int     (*v_pre)(PRE_ARGS);
Line 268  const struct valids mdoc_valids[MDOC_MAX] = {
Line 268  const struct valids mdoc_valids[MDOC_MAX] = {
         { NULL, posts_notext },                 /* br */          { NULL, posts_notext },                 /* br */
         { NULL, posts_sp },                     /* sp */          { NULL, posts_sp },                     /* sp */
         { NULL, posts_text1 },                  /* %U */          { NULL, posts_text1 },                  /* %U */
           { NULL, NULL },                         /* Ta */
 };  };
   
   
 int  int
 mdoc_valid_pre(struct mdoc *mdoc, const struct mdoc_node *n)  mdoc_valid_pre(struct mdoc *mdoc, struct mdoc_node *n)
 {  {
         v_pre           *p;          v_pre           *p;
         int              line, pos;          int              line, pos;
Line 534  pre_display(PRE_ARGS)
Line 535  pre_display(PRE_ARGS)
 static int  static int
 pre_bl(PRE_ARGS)  pre_bl(PRE_ARGS)
 {  {
         int              pos, type, width, offset;          int              i, width, offs, cmpt, dupl;
           enum mdoc_list   lt;
   
         if (MDOC_BLOCK != n->type)          if (MDOC_BLOCK != n->type) {
                   assert(n->parent);
                   assert(MDOC_BLOCK == n->parent->type);
                   assert(MDOC_Bl == n->parent->tok);
                   assert(LIST__NONE != n->parent->data.list);
                   n->data.list = n->parent->data.list;
                 return(1);                  return(1);
         if (NULL == n->args) {  
                 mdoc_nmsg(mdoc, n, MANDOCERR_LISTTYPE);  
                 return(0);  
         }          }
   
         /* Make sure that only one type of list is specified.  */          /*
            * First figure out which kind of list to use: bind ourselves to
            * the first mentioned list type and warn about any remaining
            * ones.  If we find no list type, we default to LIST_item.
            */
   
         type = offset = width = -1;          assert(LIST__NONE == n->data.list);
           offs = width = cmpt = -1;
   
         /* LINTED */          /* LINTED */
         for (pos = 0; pos < (int)n->args->argc; pos++)          for (i = 0; n->args && i < (int)n->args->argc; i++) {
                 switch (n->args->argv[pos].arg) {                  lt = LIST__NONE;
                   dupl = 0;
                   switch (n->args->argv[i].arg) {
                   /* Set list types. */
                 case (MDOC_Bullet):                  case (MDOC_Bullet):
                         /* FALLTHROUGH */                          lt = LIST_bullet;
                           break;
                 case (MDOC_Dash):                  case (MDOC_Dash):
                         /* FALLTHROUGH */                          lt = LIST_dash;
                           break;
                 case (MDOC_Enum):                  case (MDOC_Enum):
                         /* FALLTHROUGH */                          lt = LIST_enum;
                           break;
                 case (MDOC_Hyphen):                  case (MDOC_Hyphen):
                         /* FALLTHROUGH */                          lt = LIST_hyphen;
                           break;
                 case (MDOC_Item):                  case (MDOC_Item):
                         /* FALLTHROUGH */                          lt = LIST_item;
                           break;
                 case (MDOC_Tag):                  case (MDOC_Tag):
                         /* FALLTHROUGH */                          lt = LIST_tag;
                           break;
                 case (MDOC_Diag):                  case (MDOC_Diag):
                         /* FALLTHROUGH */                          lt = LIST_diag;
                           break;
                 case (MDOC_Hang):                  case (MDOC_Hang):
                         /* FALLTHROUGH */                          lt = LIST_hang;
                           break;
                 case (MDOC_Ohang):                  case (MDOC_Ohang):
                         /* FALLTHROUGH */                          lt = LIST_ohang;
                           break;
                 case (MDOC_Inset):                  case (MDOC_Inset):
                         /* FALLTHROUGH */                          lt = LIST_inset;
                           break;
                 case (MDOC_Column):                  case (MDOC_Column):
                         if (type < 0) {                          lt = LIST_column;
                                 type = n->args->argv[pos].arg;                          break;
                                 break;                  /* Set list arguments. */
                         }  
                         if (mdoc_nmsg(mdoc, n, MANDOCERR_LISTREP))  
                                 break;  
                         return(0);  
                 case (MDOC_Compact):                  case (MDOC_Compact):
                         if (type >= 0)                          if (cmpt >= 0)
                                 break;                                  dupl++;
                         if (mdoc_nmsg(mdoc, n, MANDOCERR_LISTFIRST))                          cmpt = i;
                                 break;                          break;
                         return(0);  
                 case (MDOC_Width):                  case (MDOC_Width):
                         if (width >= 0)                          if (width >= 0)
                                 if ( ! mdoc_nmsg(mdoc, n, MANDOCERR_ARGVREP))                                  dupl++;
                                         return(0);                          width = i;
                         if (type < 0 && ! mdoc_nmsg(mdoc, n, MANDOCERR_LISTFIRST))  
                                 return(0);  
                         width = n->args->argv[pos].arg;  
                         break;                          break;
                 case (MDOC_Offset):                  case (MDOC_Offset):
                         if (offset >= 0)                          if (offs >= 0)
                                 if ( ! mdoc_nmsg(mdoc, n, MANDOCERR_ARGVREP))                                  dupl++;
                                         return(0);                          offs = i;
                         if (type < 0 && ! mdoc_nmsg(mdoc, n, MANDOCERR_LISTFIRST))  
                                 return(0);  
                         offset = n->args->argv[pos].arg;  
                         break;                          break;
                 default:  
                         break;  
                 }                  }
   
         if (type < 0) {                  /* Check: duplicate auxiliary arguments. */
                 mdoc_nmsg(mdoc, n, MANDOCERR_LISTTYPE);  
                 return(0);                  if (dupl)
                           if ( ! mdoc_nmsg(mdoc, n, MANDOCERR_ARGVREP))
                                   return(0);
   
                   /* Check: multiple list types. */
   
                   if (LIST__NONE != lt && n->data.list != LIST__NONE)
                           if ( ! mdoc_nmsg(mdoc, n, MANDOCERR_LISTREP))
                                   return(0);
   
                   /* Assign list type. */
   
                   if (LIST__NONE != lt && n->data.list == LIST__NONE)
                           n->data.list = lt;
   
                   /* The list type should come first. */
   
                   if (n->data.list == LIST__NONE)
                           if (width >= 0 || offs >= 0 || cmpt >= 0)
                                   if ( ! mdoc_nmsg(mdoc, n, MANDOCERR_LISTFIRST))
                                           return(0);
   
                   continue;
         }          }
   
           /* Allow lists to default to LIST_item. */
   
           if (LIST__NONE == n->data.list) {
                   if ( ! mdoc_nmsg(mdoc, n, MANDOCERR_LISTTYPE))
                           return(0);
                   n->data.list = LIST_item;
           }
   
         /*          /*
          * Validate the width field.  Some list types don't need width           * Validate the width field.  Some list types don't need width
          * types and should be warned about them.  Others should have it           * types and should be warned about them.  Others should have it
          * and must also be warned.           * and must also be warned.
          */           */
   
         switch (type) {          switch (n->data.list) {
         case (MDOC_Tag):          case (LIST_tag):
                 if (width < 0 && ! mdoc_nmsg(mdoc, n, MANDOCERR_NOWIDTHARG))                  if (width >= 0)
                         return(0);                          break;
                 break;                  if (mdoc_nmsg(mdoc, n, MANDOCERR_NOWIDTHARG))
         case (MDOC_Column):                          break;
                   return(0);
           case (LIST_column):
                 /* FALLTHROUGH */                  /* FALLTHROUGH */
         case (MDOC_Diag):          case (LIST_diag):
                 /* FALLTHROUGH */                  /* FALLTHROUGH */
         case (MDOC_Ohang):          case (LIST_ohang):
                 /* FALLTHROUGH */                  /* FALLTHROUGH */
         case (MDOC_Inset):          case (LIST_inset):
                 /* FALLTHROUGH */                  /* FALLTHROUGH */
         case (MDOC_Item):          case (LIST_item):
                 if (width >= 0 && ! mdoc_nmsg(mdoc, n, MANDOCERR_WIDTHARG))                  if (width < 0)
                         return(0);                          break;
                 break;                  if (mdoc_nmsg(mdoc, n, MANDOCERR_WIDTHARG))
                           break;
                   return(0);
         default:          default:
                 break;                  break;
         }          }
Line 928  post_an(POST_ARGS)
Line 971  post_an(POST_ARGS)
 static int  static int
 post_it(POST_ARGS)  post_it(POST_ARGS)
 {  {
         /* FIXME: use mdoc_list! */          int               i, cols, rc;
         int               type, i, cols;          enum mdoc_list    lt;
         struct mdoc_node *n, *c;          struct mdoc_node *n, *c;
           enum mandocerr    er;
   
         if (MDOC_BLOCK != mdoc->last->type)          if (MDOC_BLOCK != mdoc->last->type)
                 return(1);                  return(1);
   
         n = mdoc->last->parent->parent;          n = mdoc->last->parent->parent;
         if (NULL == n->args) {          lt = n->data.list;
                 mdoc_nmsg(mdoc, mdoc->last, MANDOCERR_LISTTYPE);  
                 return(0);  
         }  
   
         /* Some types require block-head, some not. */          if (LIST__NONE == lt) {
   
         /* LINTED */  
         for (cols = type = -1, i = 0; -1 == type &&  
                         i < (int)n->args->argc; i++)  
                 switch (n->args->argv[i].arg) {  
                 case (MDOC_Tag):  
                         /* FALLTHROUGH */  
                 case (MDOC_Diag):  
                         /* FALLTHROUGH */  
                 case (MDOC_Hang):  
                         /* FALLTHROUGH */  
                 case (MDOC_Ohang):  
                         /* FALLTHROUGH */  
                 case (MDOC_Inset):  
                         /* FALLTHROUGH */  
                 case (MDOC_Bullet):  
                         /* FALLTHROUGH */  
                 case (MDOC_Dash):  
                         /* FALLTHROUGH */  
                 case (MDOC_Enum):  
                         /* FALLTHROUGH */  
                 case (MDOC_Hyphen):  
                         /* FALLTHROUGH */  
                 case (MDOC_Item):  
                         type = n->args->argv[i].arg;  
                         break;  
                 case (MDOC_Column):  
                         type = n->args->argv[i].arg;  
                         cols = (int)n->args->argv[i].sz;  
                         break;  
                 default:  
                         break;  
                 }  
   
         if (-1 == type) {  
                 mdoc_nmsg(mdoc, mdoc->last, MANDOCERR_LISTTYPE);                  mdoc_nmsg(mdoc, mdoc->last, MANDOCERR_LISTTYPE);
                 return(0);                  return(0);
         }          }
   
         switch (type) {          switch (lt) {
         case (MDOC_Tag):          case (LIST_tag):
                 if (NULL == mdoc->last->head->child)                  if (mdoc->last->head->child)
                         if ( ! mdoc_nmsg(mdoc, mdoc->last, MANDOCERR_NOARGS))                          break;
                                 return(0);                  /* FIXME: give this a dummy value. */
                   if ( ! mdoc_nmsg(mdoc, mdoc->last, MANDOCERR_NOARGS))
                           return(0);
                 break;                  break;
         case (MDOC_Hang):          case (LIST_hang):
                 /* FALLTHROUGH */                  /* FALLTHROUGH */
         case (MDOC_Ohang):          case (LIST_ohang):
                 /* FALLTHROUGH */                  /* FALLTHROUGH */
         case (MDOC_Inset):          case (LIST_inset):
                 /* FALLTHROUGH */                  /* FALLTHROUGH */
         case (MDOC_Diag):          case (LIST_diag):
                 if (NULL == mdoc->last->head->child)                  if (NULL == mdoc->last->head->child)
                         if ( ! mdoc_nmsg(mdoc, mdoc->last, MANDOCERR_NOARGS))                          if ( ! mdoc_nmsg(mdoc, mdoc->last, MANDOCERR_NOARGS))
                                 return(0);                                  return(0);
Line 1001  post_it(POST_ARGS)
Line 1009  post_it(POST_ARGS)
                         if ( ! mdoc_nmsg(mdoc, mdoc->last, MANDOCERR_NOBODY))                          if ( ! mdoc_nmsg(mdoc, mdoc->last, MANDOCERR_NOBODY))
                                 return(0);                                  return(0);
                 break;                  break;
         case (MDOC_Bullet):          case (LIST_bullet):
                 /* FALLTHROUGH */                  /* FALLTHROUGH */
         case (MDOC_Dash):          case (LIST_dash):
                 /* FALLTHROUGH */                  /* FALLTHROUGH */
         case (MDOC_Enum):          case (LIST_enum):
                 /* FALLTHROUGH */                  /* FALLTHROUGH */
         case (MDOC_Hyphen):          case (LIST_hyphen):
                 /* FALLTHROUGH */                  /* FALLTHROUGH */
         case (MDOC_Item):          case (LIST_item):
                 if (mdoc->last->head->child)                  if (mdoc->last->head->child)
                         if ( ! mdoc_nmsg(mdoc, mdoc->last, MANDOCERR_ARGSLOST))                          if ( ! mdoc_nmsg(mdoc, mdoc->last, MANDOCERR_ARGSLOST))
                                 return(0);                                  return(0);
Line 1017  post_it(POST_ARGS)
Line 1025  post_it(POST_ARGS)
                         if ( ! mdoc_nmsg(mdoc, mdoc->last, MANDOCERR_NOBODY))                          if ( ! mdoc_nmsg(mdoc, mdoc->last, MANDOCERR_NOBODY))
                                 return(0);                                  return(0);
                 break;                  break;
         case (MDOC_Column):          case (LIST_column):
                   cols = -1;
                   for (i = 0; i < (int)n->args->argc; i++)
                           if (MDOC_Column == n->args->argv[i].arg) {
                                   cols = (int)n->args->argv[i].sz;
                                   break;
                           }
   
                   assert(-1 != cols);
                 assert(NULL == mdoc->last->head->child);                  assert(NULL == mdoc->last->head->child);
   
                 if (NULL == mdoc->last->body->child)                  if (NULL == mdoc->last->body->child)
                         if ( ! mdoc_nmsg(mdoc, mdoc->last, MANDOCERR_NOBODY))                          if ( ! mdoc_nmsg(mdoc, mdoc->last, MANDOCERR_NOBODY))
                                 return(0);                                  return(0);
   
                 /* Count up the number of columns.  */                  for (i = 0, c = mdoc->last->child; c; c = c->next)
                 c = mdoc->last->child;  
                 for (i = 0; c; c = c->next)  
                         if (MDOC_BODY == c->type)                          if (MDOC_BODY == c->type)
                                 i++;                                  i++;
   
                 if (i < cols) {                  if (i < cols)
                         if ( ! mdoc_vmsg(mdoc, MANDOCERR_ARGCOUNT,                          er = MANDOCERR_ARGCOUNT;
                                         mdoc->last->line,                  else if (i == cols || i == cols + 1)
                                         mdoc->last->pos,  
                                         "columns == %d (have %d)",  
                                         cols, i))  
                                 return(0);  
                         break;                          break;
                 } else if (i == cols || i == cols + 1)                  else
                         break;                          er = MANDOCERR_SYNTARGCOUNT;
   
                 mdoc_vmsg(mdoc, MANDOCERR_SYNTARGCOUNT,                  rc = mdoc_vmsg(mdoc, er,
                                 mdoc->last->line, mdoc->last->pos,                                  mdoc->last->line, mdoc->last->pos,
                                 "columns == %d (have %d)", cols, i);                                  "columns == %d (have %d)", cols, i);
                 return(0);                  return(rc);
         default:          default:
                 break;                  break;
         }          }
Line 1055  post_it(POST_ARGS)
Line 1066  post_it(POST_ARGS)
 static int  static int
 post_bl_head(POST_ARGS)  post_bl_head(POST_ARGS)
 {  {
         int                     i;          int               i;
         const struct mdoc_node *n;          struct mdoc_node *n;
         const struct mdoc_argv *a;  
   
           assert(mdoc->last->parent);
         n = mdoc->last->parent;          n = mdoc->last->parent;
         assert(n->args);  
   
         for (i = 0; i < (int)n->args->argc; i++) {          if (LIST_column == n->data.list) {
                 a = &n->args->argv[i];                  for (i = 0; i < (int)n->args->argc; i++)
                 if (a->arg == MDOC_Column) {                          if (MDOC_Column == n->args->argv[i].arg)
                         if (a->sz && mdoc->last->nchild)                                  break;
                                 return(mdoc_nmsg(mdoc, n, MANDOCERR_COLUMNS));                  assert(i < (int)n->args->argc);
                         return(1);  
                   if (n->args->argv[i].sz && mdoc->last->nchild) {
                           mdoc_nmsg(mdoc, n, MANDOCERR_COLUMNS);
                           return(0);
                 }                  }
                   return(1);
         }          }
   
         if (0 == (i = mdoc->last->nchild))          if (0 == (i = mdoc->last->nchild))

Legend:
Removed from v.1.87  
changed lines
  Added in v.1.91

CVSweb