]> git.cameronkatri.com Git - mandoc.git/blobdiff - tbl_data.c
Modernize an example showing antiquated syntax,
[mandoc.git] / tbl_data.c
index 70da2311d48688d8bf208c962b3d97e31ea3acc2..40b756a05e1aace8082f9eea83c898abf6e7a148 100644 (file)
@@ -1,7 +1,7 @@
-/*     $Id: tbl_data.c,v 1.22 2011/02/06 22:02:58 kristaps Exp $ */
+/*     $Id: tbl_data.c,v 1.41 2015/10/06 18:32:20 schwarze Exp $ */
 /*
  * Copyright (c) 2009, 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv>
- * Copyright (c) 2011 Ingo Schwarze <schwarze@openbsd.org>
+ * Copyright (c) 2011, 2015 Ingo Schwarze <schwarze@openbsd.org>
  *
  * Permission to use, copy, modify, and distribute this software for any
  * purpose with or without fee is hereby granted, provided that the above
@@ -15,9 +15,9 @@
  * ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
  * OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
  */
-#ifdef HAVE_CONFIG_H
 #include "config.h"
-#endif
+
+#include <sys/types.h>
 
 #include <assert.h>
 #include <ctype.h>
 #include <time.h>
 
 #include "mandoc.h"
+#include "mandoc_aux.h"
 #include "libmandoc.h"
 #include "libroff.h"
 
-static int              data(struct tbl_node *, struct tbl_span *, 
+static void             getdata(struct tbl_node *, struct tbl_span *,
                                int, const char *, int *);
-static struct tbl_span *newspan(struct tbl_node *, int, 
+static struct tbl_span *newspan(struct tbl_node *, int,
                                struct tbl_row *);
 
-static int
-data(struct tbl_node *tbl, struct tbl_span *dp, 
+
+static void
+getdata(struct tbl_node *tbl, struct tbl_span *dp,
                int ln, const char *p, int *pos)
 {
        struct tbl_dat  *dat;
        struct tbl_cell *cp;
-       int              sv, spans;
+       int              sv;
 
-       cp = NULL;
-       if (dp->last && dp->last->layout)
-               cp = dp->last->layout->next;
-       else if (NULL == dp->last)
-               cp = dp->layout->first;
+       /* Advance to the next layout cell, skipping spanners. */
 
-       /* 
-        * Skip over spanners and vertical lines to data formats, since
-        * we want to match data with data layout cells in the header.
-        */
-
-       while (cp && (TBL_CELL_VERT == cp->pos || 
-                               TBL_CELL_DVERT == cp->pos ||
-                               TBL_CELL_SPAN == cp->pos))
+       cp = dp->last == NULL ? dp->layout->first : dp->last->layout->next;
+       while (cp != NULL && cp->pos == TBL_CELL_SPAN)
                cp = cp->next;
 
        /*
@@ -63,33 +55,30 @@ data(struct tbl_node *tbl, struct tbl_span *dp,
         * cells.  This means that we have extra input.
         */
 
-       if (NULL == cp) {
-               TBL_MSG(tbl, MANDOCERR_TBLEXTRADAT, ln, *pos);
+       if (cp == NULL) {
+               mandoc_msg(MANDOCERR_TBLDATA_EXTRA, tbl->parse,
+                   ln, *pos, p + *pos);
                /* Skip to the end... */
                while (p[*pos])
                        (*pos)++;
-               return(1);
+               return;
        }
 
-       dat = mandoc_calloc(1, sizeof(struct tbl_dat));
+       dat = mandoc_calloc(1, sizeof(*dat));
        dat->layout = cp;
        dat->pos = TBL_DATA_NONE;
-
-       assert(TBL_CELL_SPAN != cp->pos);
-
-       for (spans = 0, cp = cp->next; cp; cp = cp->next)
-               if (TBL_CELL_SPAN == cp->pos)
-                       spans++;
+       dat->spans = 0;
+       for (cp = cp->next; cp != NULL; cp = cp->next)
+               if (cp->pos == TBL_CELL_SPAN)
+                       dat->spans++;
                else
                        break;
-       
-       dat->spans = spans;
 
-       if (dp->last) {
+       if (dp->last == NULL)
+               dp->first = dat;
+       else
                dp->last->next = dat;
-               dp->last = dat;
-       } else
-               dp->last = dp->first = dat;
+       dp->last = dat;
 
        sv = *pos;
        while (p[*pos] && p[*pos] != tbl->opts.tab)
@@ -101,14 +90,12 @@ data(struct tbl_node *tbl, struct tbl_span *dp,
         * until a standalone `T}', are included in our cell.
         */
 
-       if (*pos - sv == 2 && 'T' == p[sv] && '{' == p[sv + 1]) {
+       if (*pos - sv == 2 && p[sv] == 'T' && p[sv + 1] == '{') {
                tbl->part = TBL_PART_CDATA;
-               return(0);
+               return;
        }
 
-       dat->string = mandoc_malloc(*pos - sv + 1);
-       memcpy(dat->string, &p[sv], *pos - sv);
-       dat->string[*pos - sv] = '\0';
+       dat->string = mandoc_strndup(p + sv, *pos - sv);
 
        if (p[*pos])
                (*pos)++;
@@ -124,24 +111,19 @@ data(struct tbl_node *tbl, struct tbl_span *dp,
        else
                dat->pos = TBL_DATA_DATA;
 
-       if (TBL_CELL_HORIZ == dat->layout->pos ||
-                       TBL_CELL_DHORIZ == dat->layout->pos ||
-                       TBL_CELL_DOWN == dat->layout->pos)
-               if (TBL_DATA_DATA == dat->pos && '\0' != *dat->string)
-                       TBL_MSG(tbl, MANDOCERR_TBLIGNDATA, ln, sv);
-
-       return(1);
+       if ((dat->layout->pos == TBL_CELL_HORIZ ||
+           dat->layout->pos == TBL_CELL_DHORIZ ||
+           dat->layout->pos == TBL_CELL_DOWN) &&
+           dat->pos == TBL_DATA_DATA && *dat->string != '\0')
+               mandoc_msg(MANDOCERR_TBLDATA_SPAN,
+                   tbl->parse, ln, sv, dat->string);
 }
 
-/* ARGSUSED */
 int
-tbl_cdata(struct tbl_node *tbl, int ln, const char *p)
+tbl_cdata(struct tbl_node *tbl, int ln, const char *p, int pos)
 {
        struct tbl_dat  *dat;
-       size_t           sz;
-       int              pos;
-
-       pos = 0;
+       size_t           sz;
 
        dat = tbl->last_span->last;
 
@@ -150,10 +132,12 @@ tbl_cdata(struct tbl_node *tbl, int ln, const char *p)
                if (p[pos] == tbl->opts.tab) {
                        tbl->part = TBL_PART_DATA;
                        pos++;
-                       return(data(tbl, tbl->last_span, ln, p, &pos));
-               } else if ('\0' == p[pos]) {
+                       while (p[pos] != '\0')
+                               getdata(tbl, tbl->last_span, ln, p, &pos);
+                       return 1;
+               } else if (p[pos] == '\0') {
                        tbl->part = TBL_PART_DATA;
-                       return(1);
+                       return 1;
                }
 
                /* Fallthrough: T} is part of a word. */
@@ -161,18 +145,19 @@ tbl_cdata(struct tbl_node *tbl, int ln, const char *p)
 
        dat->pos = TBL_DATA_DATA;
 
-       if (dat->string) {
-               sz = strlen(p) + strlen(dat->string) + 2;
+       if (dat->string != NULL) {
+               sz = strlen(p + pos) + strlen(dat->string) + 2;
                dat->string = mandoc_realloc(dat->string, sz);
-               strlcat(dat->string, " ", sz);
-               strlcat(dat->string, p, sz);
+               (void)strlcat(dat->string, " ", sz);
+               (void)strlcat(dat->string, p + pos, sz);
        } else
-               dat->string = mandoc_strdup(p);
+               dat->string = mandoc_strdup(p + pos);
 
-       if (TBL_CELL_DOWN == dat->layout->pos) 
-               TBL_MSG(tbl, MANDOCERR_TBLIGNDATA, ln, pos);
+       if (dat->layout->pos == TBL_CELL_DOWN)
+               mandoc_msg(MANDOCERR_TBLDATA_SPAN, tbl->parse,
+                   ln, pos, dat->string);
 
-       return(0);
+       return 0;
 }
 
 static struct tbl_span *
@@ -180,39 +165,29 @@ newspan(struct tbl_node *tbl, int line, struct tbl_row *rp)
 {
        struct tbl_span *dp;
 
-       dp = mandoc_calloc(1, sizeof(struct tbl_span));
+       dp = mandoc_calloc(1, sizeof(*dp));
        dp->line = line;
-       dp->tbl = &tbl->opts;
+       dp->opts = &tbl->opts;
        dp->layout = rp;
-       dp->head = tbl->first_head;
+       dp->prev = tbl->last_span;
 
-       if (tbl->last_span) {
-               tbl->last_span->next = dp;
-               tbl->last_span = dp;
-       } else {
-               tbl->last_span = tbl->first_span = dp;
+       if (dp->prev == NULL) {
+               tbl->first_span = dp;
                tbl->current_span = NULL;
-               dp->flags |= TBL_SPAN_FIRST;
-       }
+       } else
+               dp->prev->next = dp;
+       tbl->last_span = dp;
 
-       return(dp);
+       return dp;
 }
 
-int
-tbl_data(struct tbl_node *tbl, int ln, const char *p)
+void
+tbl_data(struct tbl_node *tbl, int ln, const char *p, int pos)
 {
        struct tbl_span *dp;
        struct tbl_row  *rp;
-       int              pos;
 
-       pos = 0;
-
-       if ('\0' == p[pos]) {
-               TBL_MSG(tbl, MANDOCERR_TBL, ln, pos);
-               return(0);
-       }
-
-       /* 
+       /*
         * Choose a layout row: take the one following the last parsed
         * span's.  If that doesn't exist, use the last parsed span's.
         * If there's no last parsed span, use the first row.  Lastly,
@@ -220,17 +195,17 @@ tbl_data(struct tbl_node *tbl, int ln, const char *p)
         * (it doesn't "consume" the layout).
         */
 
-       if (tbl->last_span) {
-               assert(tbl->last_span->layout);
+       if (tbl->last_span != NULL) {
                if (tbl->last_span->pos == TBL_SPAN_DATA) {
                        for (rp = tbl->last_span->layout->next;
-                                       rp && rp->first; rp = rp->next) {
+                            rp != NULL && rp->first != NULL;
+                            rp = rp->next) {
                                switch (rp->first->pos) {
-                               case (TBL_CELL_HORIZ):
+                               case TBL_CELL_HORIZ:
                                        dp = newspan(tbl, ln, rp);
                                        dp->pos = TBL_SPAN_HORIZ;
                                        continue;
-                               case (TBL_CELL_DHORIZ):
+                               case TBL_CELL_DHORIZ:
                                        dp = newspan(tbl, ln, rp);
                                        dp->pos = TBL_SPAN_DHORIZ;
                                        continue;
@@ -242,7 +217,7 @@ tbl_data(struct tbl_node *tbl, int ln, const char *p)
                } else
                        rp = tbl->last_span->layout;
 
-               if (NULL == rp)
+               if (rp == NULL)
                        rp = tbl->last_span->layout;
        } else
                rp = tbl->first_row;
@@ -253,19 +228,14 @@ tbl_data(struct tbl_node *tbl, int ln, const char *p)
 
        if ( ! strcmp(p, "_")) {
                dp->pos = TBL_SPAN_HORIZ;
-               return(1);
+               return;
        } else if ( ! strcmp(p, "=")) {
                dp->pos = TBL_SPAN_DHORIZ;
-               return(1);
+               return;
        }
 
        dp->pos = TBL_SPAN_DATA;
 
-       /* This returns 0 when TBL_PART_CDATA is entered. */
-
-       while ('\0' != p[pos])
-               if ( ! data(tbl, dp, ln, p, &pos))
-                       return(0);
-
-       return(1);
+       while (p[pos] != '\0')
+               getdata(tbl, dp, ln, p, &pos);
 }