The CommonMark specification allows list markers fo the form "number) "

[mandoc.git] / read.c

diff --git a/read.c b/read.c
index 21d28b0bc08f430e9a79966d420bcebf90cae9d4..883f5412750893b9e4525ffb25ec57068d3be809 100644 (file)
--- a/read.c
+++ b/read.c
@@ -1,7 +1,7 @@
-/*     $Id: read.c,v 1.151 2016/07/19 21:31:55 schwarze Exp $ */
+/*     $Id: read.c,v 1.163 2017/03/07 20:00:08 schwarze Exp $ */

 /*
  * 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-2016 Ingo Schwarze <schwarze@openbsd.org>
+ * Copyright (c) 2010-2017 Ingo Schwarze <schwarze@openbsd.org>

  * Copyright (c) 2010, 2012 Joerg Sonnenberger <joerg@netbsd.org>
  *
  * Permission to use, copy, modify, and distribute this software for any
  * Copyright (c) 2010, 2012 Joerg Sonnenberger <joerg@netbsd.org>
  *
  * Permission to use, copy, modify, and distribute this software for any


@@ -30,7 +30,6 @@
 #include <errno.h>
 #include <fcntl.h>
 #include <stdarg.h>
 #include <errno.h>
 #include <fcntl.h>
 #include <stdarg.h>

-#include <stdint.h>

 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>


@@ -48,8 +47,8 @@
 #define        REPARSE_LIMIT   1000
 
 struct mparse {
 #define        REPARSE_LIMIT   1000
 
 struct mparse {

-       struct roff_man  *man; /* man parser */

        struct roff      *roff; /* roff parser (!NULL) */
        struct roff      *roff; /* roff parser (!NULL) */

+       struct roff_man  *man; /* man parser */

        char             *sodest; /* filename pointed to by .so */
        const char       *file; /* filename of current input file */
        struct buf       *primary; /* buffer currently being parsed */
        char             *sodest; /* filename pointed to by .so */
        const char       *file; /* filename of current input file */
        struct buf       *primary; /* buffer currently being parsed */


@@ -67,7 +66,7 @@ struct        mparse {
 
 static void      choose_parser(struct mparse *);
 static void      resize_buf(struct buf *, size_t);
 
 static void      choose_parser(struct mparse *);
 static void      resize_buf(struct buf *, size_t);

-static void      mparse_buf_r(struct mparse *, struct buf, size_t, int);
+static int       mparse_buf_r(struct mparse *, struct buf, size_t, int);

 static int       read_whole_file(struct mparse *, const char *, int,
                                struct buf *, int *);
 static void      mparse_end(struct mparse *);
 static int       read_whole_file(struct mparse *, const char *, int,
                                struct buf *, int *);
 static void      mparse_end(struct mparse *);


@@ -108,11 +107,13 @@ static    const char * const      mandocerrs[MANDOCERR_MAX] = {
        "no document body",
        "content before first section header",
        "first section is not \"NAME\"",
        "no document body",
        "content before first section header",
        "first section is not \"NAME\"",

-       "NAME section without name",
+       "NAME section without Nm before Nd",

        "NAME section without description",
        "description not at the end of NAME",
        "bad NAME section content",
        "NAME section without description",
        "description not at the end of NAME",
        "bad NAME section content",

+       "missing comma before name",

        "missing description line, using \"\"",
        "missing description line, using \"\"",

+       "description line outside NAME section",

        "sections out of conventional order",
        "duplicate section title",
        "unexpected section",
        "sections out of conventional order",
        "duplicate section title",
        "unexpected section",


@@ -141,7 +142,7 @@ static      const char * const      mandocerrs[MANDOCERR_MAX] = {
        "empty argument, using 0n",
        "missing display type, using -ragged",
        "list type is not the first argument",
        "empty argument, using 0n",
        "missing display type, using -ragged",
        "list type is not the first argument",

-       "missing -width in -tag list, using 8n",
+       "missing -width in -tag list, using 6n",

        "missing utility name, using \"\"",
        "missing function name, using \"\"",
        "empty head in list item",
        "missing utility name, using \"\"",
        "missing function name, using \"\"",
        "empty head in list item",


@@ -150,6 +151,7 @@ static      const char * const      mandocerrs[MANDOCERR_MAX] = {
        "unknown font type, using \\fR",
        "nothing follows prefix",
        "empty reference block",
        "unknown font type, using \\fR",
        "nothing follows prefix",
        "empty reference block",

+       "missing section argument",

        "missing -std argument, adding it",
        "missing option string, using \"\"",
        "missing resource identifier, using \"\"",
        "missing -std argument, adding it",
        "missing option string, using \"\"",
        "missing resource identifier, using \"\"",


@@ -175,6 +177,7 @@ static      const char * const      mandocerrs[MANDOCERR_MAX] = {
        "blank line in fill mode, using .sp",
        "tab in filled text",
        "whitespace at end of input line",
        "blank line in fill mode, using .sp",
        "tab in filled text",
        "whitespace at end of input line",

+       "new sentence, new line",

        "bad comment style",
        "invalid escape sequence",
        "undefined string, using \"\"",
        "bad comment style",
        "invalid escape sequence",
        "undefined string, using \"\"",


@@ -307,7 +310,7 @@ choose_parser(struct mparse *curp)
  * macros, inline equations, and input line traps)
  * and indirectly (for .so file inclusion).
  */
  * macros, inline equations, and input line traps)
  * and indirectly (for .so file inclusion).
  */

-static void
+static int

 mparse_buf_r(struct mparse *curp, struct buf blk, size_t i, int start)
 {
        const struct tbl_span   *span;
 mparse_buf_r(struct mparse *curp, struct buf blk, size_t i, int start)
 {
        const struct tbl_span   *span;


@@ -315,6 +318,7 @@ mparse_buf_r(struct mparse *curp, struct buf blk, size_t i, int start)
        const char      *save_file;
        char            *cp;
        size_t           pos; /* byte number in the ln buffer */
        const char      *save_file;
        char            *cp;
        size_t           pos; /* byte number in the ln buffer */

+       size_t           j;  /* auxiliary byte number in the blk buffer */

        enum rofferr     rr;
        int              of;
        int              lnn; /* line number in the real file */
        enum rofferr     rr;
        int              of;
        int              lnn; /* line number in the real file */


@@ -420,14 +424,21 @@ mparse_buf_r(struct mparse *curp, struct buf blk, size_t i, int start)
                        }
 
                        if ('"' == blk.buf[i + 1] || '#' == blk.buf[i + 1]) {
                        }
 
                        if ('"' == blk.buf[i + 1] || '#' == blk.buf[i + 1]) {

+                               j = i;

                                i += 2;
                                /* Comment, skip to end of line */
                                for (; i < blk.sz; ++i) {
                                i += 2;
                                /* Comment, skip to end of line */
                                for (; i < blk.sz; ++i) {

-                                       if ('\n' == blk.buf[i]) {
-                                               ++i;
-                                               ++lnn;
-                                               break;
-                                       }
+                                       if (blk.buf[i] != '\n')
+                                               continue;
+                                       if (blk.buf[i - 1] == ' ' ||
+                                           blk.buf[i - 1] == '\t')
+                                               mandoc_msg(
+                                                   MANDOCERR_SPACE_EOL,
+                                                   curp, curp->line,
+                                                   pos + i-1 - j, NULL);
+                                       ++i;
+                                       ++lnn;
+                                       break;

                                }
 
                                /* Backout trailing whitespaces */
                                }
 
                                /* Backout trailing whitespaces */


@@ -500,13 +511,16 @@ rerun:
 
                switch (rr) {
                case ROFF_REPARSE:
 
                switch (rr) {
                case ROFF_REPARSE:

-                       if (REPARSE_LIMIT >= ++curp->reparse_count)
-                               mparse_buf_r(curp, ln, of, 0);
-                       else
+                       if (++curp->reparse_count > REPARSE_LIMIT)

                                mandoc_msg(MANDOCERR_ROFFLOOP, curp,
                                    curp->line, pos, NULL);
                                mandoc_msg(MANDOCERR_ROFFLOOP, curp,
                                    curp->line, pos, NULL);

-                       pos = 0;
-                       continue;
+                       else if (mparse_buf_r(curp, ln, of, 0) == 1 ||
+                           start == 1) {
+                               pos = 0;
+                               continue;
+                       }
+                       free(ln.buf);
+                       return 0;

                case ROFF_APPEND:
                        pos = strlen(ln.buf);
                        continue;
                case ROFF_APPEND:
                        pos = strlen(ln.buf);
                        continue;


@@ -520,7 +534,7 @@ rerun:
                            (i >= blk.sz || blk.buf[i] == '\0')) {
                                curp->sodest = mandoc_strdup(ln.buf + of);
                                free(ln.buf);
                            (i >= blk.sz || blk.buf[i] == '\0')) {
                                curp->sodest = mandoc_strdup(ln.buf + of);
                                free(ln.buf);

-                               return;
+                               return 1;

                        }
                        /*
                         * We remove `so' clauses from our lookaside
                        }
                        /*
                         * We remove `so' clauses from our lookaside


@@ -586,18 +600,18 @@ rerun:
        }
 
        free(ln.buf);
        }
 
        free(ln.buf);

+       return 1;

 }
 
 static int
 read_whole_file(struct mparse *curp, const char *file, int fd,
                struct buf *fb, int *with_mmap)
 {
 }
 
 static int
 read_whole_file(struct mparse *curp, const char *file, int fd,
                struct buf *fb, int *with_mmap)
 {

+       struct stat      st;

        gzFile           gz;
        size_t           off;
        ssize_t          ssz;
 
        gzFile           gz;
        size_t           off;
        ssize_t          ssz;
 

-       struct stat      st;
-

        if (fstat(fd, &st) == -1)
                err((int)MANDOCLEVEL_SYSERR, "%s", file);
 
        if (fstat(fd, &st) == -1)
                err((int)MANDOCLEVEL_SYSERR, "%s", file);
 


@@ -819,13 +833,15 @@ mparse_reset(struct mparse *curp)
 {
        roff_reset(curp->roff);
        roff_man_reset(curp->man);
 {
        roff_reset(curp->roff);
        roff_man_reset(curp->man);

+
+       free(curp->sodest);
+       curp->sodest = NULL;
+

        if (curp->secondary)
                curp->secondary->sz = 0;
 
        curp->file_status = MANDOCLEVEL_OK;
        if (curp->secondary)
                curp->secondary->sz = 0;
 
        curp->file_status = MANDOCLEVEL_OK;

-
-       free(curp->sodest);
-       curp->sodest = NULL;
+       curp->gzip = 0;

 }
 
 void
 }
 
 void


@@ -833,8 +849,7 @@ mparse_free(struct mparse *curp)
 {
 
        roff_man_free(curp->man);
 {
 
        roff_man_free(curp->man);

-       if (curp->roff)
-               roff_free(curp->roff);
+       roff_free(curp->roff);

        if (curp->secondary)
                free(curp->secondary->buf);
 
        if (curp->secondary)
                free(curp->secondary->buf);
 


@@ -856,6 +871,13 @@ mparse_result(struct mparse *curp, struct roff_man **man,
                *man = curp->man;
 }
 
                *man = curp->man;
 }
 

+void
+mparse_updaterc(struct mparse *curp, enum mandoclevel *rc)
+{
+       if (curp->file_status > *rc)
+               *rc = curp->file_status;
+}
+

 void
 mandoc_vmsg(enum mandocerr t, struct mparse *m,
                int ln, int pos, const char *fmt, ...)
 void
 mandoc_vmsg(enum mandocerr t, struct mparse *m,
                int ln, int pos, const char *fmt, ...)