From: schwarze <schwarze@openbsd.org>
Date: Sat, 17 Jun 2017 19:17:41 +0000 (+0000)
Subject: delete the -d option and the "use .Fn or .Xr for functions" check,
X-Git-Url: http://artulab.com/gitweb/?a=commitdiff_plain;h=3bfe4a650f53d3e576dc8f805dc5d6b77d6158cf;p=openbsd

delete the -d option and the "use .Fn or .Xr for functions" check,
both covered by mandoc now; OK jmc@
---

diff --git a/regress/usr.bin/mdoclint/mdoclint b/regress/usr.bin/mdoclint/mdoclint
index 2653df122a4..a5e6ec34959 100644
--- a/regress/usr.bin/mdoclint/mdoclint
+++ b/regress/usr.bin/mdoclint/mdoclint
@@ -1,6 +1,6 @@
 #!/usr/bin/perl
 #
-# $OpenBSD: mdoclint,v 1.68 2017/06/08 18:25:16 schwarze Exp $
+# $OpenBSD: mdoclint,v 1.69 2017/06/17 19:17:41 schwarze Exp $
 # $NetBSD: mdoclint,v 1.77 2017/06/08 10:19:56 wiz Exp $
 #
 # Copyright (c) 2001-2017 Thomas Klausner
@@ -42,14 +42,14 @@ use constant {
 };
 
 use vars qw(
-	$opt_D $opt_d $opt_F $opt_h
+	$opt_D $opt_F $opt_h
 	$opt_m $opt_o $opt_p $opt_r $opt_S $opt_v $opt_w $opt_x
 );
 
 
 my $arch=`uname -m`;
 chomp($arch);
-my $options="DdFhmoprSvwx";
+my $options="DFhmoprSvwx";
 
 sub usage
 {
@@ -57,7 +57,6 @@ sub usage
 mdoclint: verify man page correctness
 usage:	mdoclint [-$options] file ...
 	-D	warn about bad casing and archs in .Dt
-	-d	warn about bad date strings (in .Dd only)
 	-F	fix whitespace problems (asks before overwriting)
 	-h	display this help text
 	-m	warn about man pages that are not in mdoc(7) format
@@ -68,7 +67,7 @@ usage:	mdoclint [-$options] file ...
 	-v	verbose output
 	-w	show section header in warnings
 	-x	warn about cross-references with missing targets
-Default is -DdmoprSx if no flag is specified.
+Default is -DmoprSx if no flag is specified.
 EOF
 	exit(0);
 }
@@ -78,7 +77,7 @@ my %sections;
 my $arches_re;
 my $sections_re;
 my $esections_re;
-my $valid_date_re;
+
 # and the code that builds them
 {
 	my @sections = (
@@ -141,12 +140,6 @@ my $valid_date_re;
 		$sections_re = qr{[1-9](?:lua)?}o;
 		$esections_re = qr{[0-9](?:lua)?}o;
 	}
-	if (OPENBSD) {
-		$valid_date_re = qr{\$Mdocdate\b};
-	}
-	if (NETBSD) {
-		$valid_date_re = qr{(?:January|February|March|April|May|June|July|August|September|October|November|December)\s*[1-9][0-9]*,\s*(?:198[0-9]|199[0-9]|200[0-9]|201[0-7])$}o;
-	}
 }
 
 sub debug
@@ -171,9 +164,9 @@ sub handle_options
 	$opt_h and usage();
 
 	# default to all warnings if no flag is set
-	unless ($opt_D or $opt_d or $opt_m
+	unless ($opt_D or $opt_m
 	    or $opt_o or $opt_p or $opt_r or $opt_S or $opt_x) {
-		$opt_D = $opt_d = $opt_m =
+		$opt_D = $opt_m =
 		    $opt_o = $opt_p = $opt_r = $opt_S = $opt_x = 1;
 	}
 }
@@ -370,9 +363,6 @@ sub process_line
 	if (/^\./o and /Ns [\.();,\[\]\{\}:]/o) {
 		$s->warning("possible Ns abuse: `$_'") if $opt_p;
 	}
-	if ((/^([^\.]\w+)\(\)/o or /^[^\.].*\s(\w+)\(\)/o) and not $s->{inliteral}) {
-		$s->warning("use .Fn or .Xr for functions: `$1()'") if $opt_p;
-	}
 
 	my $destruct = $_;
 	if ($s->{mandoc_p}) {
@@ -398,11 +388,6 @@ sub process_line
 			}
 		}
 	}
-
-	if (/^\.Dd/o and not /^\.Dd\s+$valid_date_re/o) {
-		$s->warning("Invalid date found: `$_'") if $opt_d;
-	}
-
 	if (/^\.Bd\b.*-(?:literal|unfilled)\b/o) {
 		$s->{inliteral} = 1;
 	}
diff --git a/regress/usr.bin/mdoclint/mdoclint.1 b/regress/usr.bin/mdoclint/mdoclint.1
index 9f268411253..32578a90cfc 100644
--- a/regress/usr.bin/mdoclint/mdoclint.1
+++ b/regress/usr.bin/mdoclint/mdoclint.1
@@ -1,4 +1,4 @@
-.\" $OpenBSD: mdoclint.1,v 1.26 2017/06/08 18:25:16 schwarze Exp $
+.\" $OpenBSD: mdoclint.1,v 1.27 2017/06/17 19:17:41 schwarze Exp $
 .\" $NetBSD: mdoclint.1,v 1.23 2017/06/08 10:19:56 wiz Exp $
 .\"
 .\" Copyright (c) 2001-2013 Thomas Klausner
@@ -25,7 +25,7 @@
 .\" ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
 .\" POSSIBILITY OF SUCH DAMAGE.
 .\"
-.Dd $Mdocdate: June 8 2017 $
+.Dd $Mdocdate: June 17 2017 $
 .Dt MDOCLINT 1
 .Os
 .Sh NAME
@@ -33,7 +33,7 @@
 .Nd man page verifier
 .Sh SYNOPSIS
 .Nm
-.Op Fl DdFhmoprSvwx
+.Op Fl DFhmoprSvwx
 .Ar
 .Sh DESCRIPTION
 .Nm
@@ -41,7 +41,7 @@ is a man page verifier.
 It tries to automatically find as many common
 errors that occur when writing man pages as possible.
 If no flags are given,
-.Fl DdmoprSx
+.Fl DmoprSx
 is assumed (that is, everything except
 .Fl Fhvw ) .
 .Pp
@@ -49,8 +49,6 @@ The options are as follows:
 .Bl -tag -width Ds
 .It Fl D
 Warn about bad casing and architectures in the .Dt macro.
-.It Fl d
-Warn about bad date strings (in the .Dd macro).
 .It Fl F
 Delete whitespace at the end of input lines.
 .It Fl h