style message about missing .Fn markup; inspired by mdoclint

diff --git a/mandoc.1 b/mandoc.1
index 3cbe4561c589e14d5c944c887182d2c1a7ddb27d..1fe4603207282bf73fdf2a1d67b898f048bc5d23 100644 (file)
--- a/mandoc.1
+++ b/mandoc.1
@@ -1,4 +1,4 @@
-.\"    $Id: mandoc.1,v 1.197 2017/06/10 01:48:53 schwarze Exp $
+.\"    $Id: mandoc.1,v 1.198 2017/06/11 17:16:51 schwarze Exp $
 .\"
 .\" Copyright (c) 2009, 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv>
 .\" Copyright (c) 2012, 2014-2017 Ingo Schwarze <schwarze@openbsd.org>
@@ -15,7 +15,7 @@
 .\" ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
 .\" OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
 .\"
-.Dd $Mdocdate: June 10 2017 $
+.Dd $Mdocdate: June 11 2017 $
 .Dt MANDOC 1
 .Os
 .Sh NAME
@@ -799,10 +799,19 @@ Do not use punctuation at the end of an
 .Ic \&Nd
 block.
 .It Sy "no blank before trailing delimiter"
+.Pq mdoc
 The last argument of a macro that supports trailing delimiter
 arguments is longer than one byte and ends with a trailing delimiter.
 Consider inserting a blank such that the delimiter becomes a separate
 argument, thus moving it out of the scope of the macro.
+.It Sy "function name without markup"
+.Pq mdoc
+A word followed by an empty pair of parentheses occurs on a text line.
+Consider using an
+.Ic \&Fn
+or
+.Ic \&Xr
+macro.
 .El
 .Ss Warnings related to the document prologue
 .Bl -ohang

diff --git a/mandoc.h b/mandoc.h
index 80a4f3c6ebd784ad0a89cadbc27328d78db9ff1b..55b88a63bede592ea6c820a97a55cce5d604a11a 100644 (file)
--- a/mandoc.h
+++ b/mandoc.h
@@ -1,4 +1,4 @@
-/*     $Id: mandoc.h,v 1.227 2017/06/10 01:48:53 schwarze Exp $ */
+/*     $Id: mandoc.h,v 1.228 2017/06/11 17:16:51 schwarze Exp $ */
 /*
  * Copyright (c) 2010, 2011, 2014 Kristaps Dzonsons <kristaps@bsd.lv>
  * Copyright (c) 2010-2017 Ingo Schwarze <schwarze@openbsd.org>
@@ -52,6 +52,7 @@ enum  mandocerr {
        MANDOCERR_ER_REP, /* duplicate errno: Er ... */
        MANDOCERR_ND_DOT, /* description line ends with a full stop */
        MANDOCERR_DELIM, /* no blank before trailing delimiter: macro ... */
+       MANDOCERR_FUNC, /* function name without markup: name() */
 
        MANDOCERR_WARNING, /* ===== start of warnings ===== */
 

diff --git a/mdoc_validate.c b/mdoc_validate.c
index 84ea630c9daee2162d56ed6d9296e5c2b8e1f551..0ba42e2a99f7f664526a182fce4a3d7ce1c678f9 100644 (file)
--- a/mdoc_validate.c
+++ b/mdoc_validate.c
@@ -1,4 +1,4 @@
-/*     $Id: mdoc_validate.c,v 1.335 2017/06/11 14:24:55 schwarze Exp $ */
+/*     $Id: mdoc_validate.c,v 1.336 2017/06/11 17:16:51 schwarze Exp $ */
 /*
  * Copyright (c) 2008-2012 Kristaps Dzonsons <kristaps@bsd.lv>
  * Copyright (c) 2010-2017 Ingo Schwarze <schwarze@openbsd.org>
@@ -53,10 +53,10 @@ typedef     void    (*v_post)(POST_ARGS);
 
 static int      build_list(struct roff_man *, int);
 static void     check_text(struct roff_man *, int, int, char *);
-static void     check_bsd(struct roff_man *, int, int, char *);
 static void     check_argv(struct roff_man *,
                        struct roff_node *, struct mdoc_argv *);
 static void     check_args(struct roff_man *, struct roff_node *);
+static void     check_toptext(struct roff_man *, int, int, const char *);
 static int      child_an(const struct roff_node *);
 static size_t          macro2len(enum roff_tok);
 static void     rewrite_macro2len(struct roff_man *, char **);
@@ -304,10 +304,11 @@ mdoc_node_validate(struct roff_man *mdoc)
                if (n->sec != SEC_SYNOPSIS ||
                    (n->parent->tok != MDOC_Cd && n->parent->tok != MDOC_Fd))
                        check_text(mdoc, n->line, n->pos, n->string);
-               if (n->parent->tok == MDOC_Sh ||
-                   n->parent->tok == MDOC_Ss ||
-                   n->parent->tok == MDOC_It)
-                       check_bsd(mdoc, n->line, n->pos, n->string);
+               if (n->parent->tok == MDOC_It ||
+                   (n->parent->type == ROFFT_BODY &&
+                    (n->parent->tok == MDOC_Sh ||
+                     n->parent->tok == MDOC_Ss)))
+                       check_toptext(mdoc, n->line, n->pos, n->string);
                break;
        case ROFFT_EQN:
        case ROFFT_TBL:
@@ -390,9 +391,12 @@ check_text(struct roff_man *mdoc, int ln, int pos, char *p)
 }
 
 static void
-check_bsd(struct roff_man *mdoc, int ln, int pos, char *p)
+check_toptext(struct roff_man *mdoc, int ln, int pos, const char *p)
 {
-       const char      *cp;
+       const char      *cp, *cpr;
+
+       if (*p == '\0')
+               return;
 
        if ((cp = strstr(p, "OpenBSD")) != NULL)
                mandoc_msg(MANDOCERR_BX, mdoc->parse,
@@ -406,6 +410,19 @@ check_bsd(struct roff_man *mdoc, int ln, int pos, char *p)
        if ((cp = strstr(p, "DragonFly")) != NULL)
                mandoc_msg(MANDOCERR_BX, mdoc->parse,
                    ln, pos + (cp - p), "Dx");
+
+       cp = p;
+       while ((cp = strstr(cp + 1, "()")) != NULL) {
+               for (cpr = cp - 1; cpr >= p; cpr--)
+                       if (*cpr != '_' && !isalnum((unsigned char)*cpr))
+                               break;
+               if ((cpr < p || *cpr == ' ') && cpr + 1 < cp) {
+                       cpr++;
+                       mandoc_vmsg(MANDOCERR_FUNC, mdoc->parse,
+                           ln, pos + (cpr - p),
+                           "%.*s()", (int)(cp - cpr), cpr);
+               }
+       }
 }
 
 static void

diff --git a/read.c b/read.c
index cd787ce84226723f002f1159cd98356115efeec7..82996eaf7f8f10a2e8e872e7ef4db52c7058f3de 100644 (file)
--- a/read.c
+++ b/read.c
@@ -1,4 +1,4 @@
-/*     $Id: read.c,v 1.174 2017/06/10 01:48:53 schwarze Exp $ */
+/*     $Id: read.c,v 1.175 2017/06/11 17:16:51 schwarze Exp $ */
 /*
  * Copyright (c) 2008, 2009, 2010, 2011 Kristaps Dzonsons <kristaps@bsd.lv>
  * Copyright (c) 2010-2017 Ingo Schwarze <schwarze@openbsd.org>
@@ -94,6 +94,7 @@ static        const char * const      mandocerrs[MANDOCERR_MAX] = {
        "duplicate errno",
        "description line ends with a full stop",
        "no blank before trailing delimiter",
+       "function name without markup",
 
        "generic warning",