remove the -s option, it is fully covered by mandoc -Tlint;

OK jmc@

2 files changed, 12 insertions, 19 deletions

diff --git a/regress/usr.bin/mdoclint/mdoclint b/regress/usr.bin/mdoclint/mdoclint
index 41acfd9bf02..a487a3f9530 100644
--- a/regress/usr.bin/mdoclint/mdoclint
+++ b/regress/usr.bin/mdoclint/mdoclint
@@ -1,6 +1,6 @@
 #!/usr/bin/perl
 #
-# $OpenBSD: mdoclint,v 1.59 2017/05/02 17:51:32 schwarze Exp $
+# $OpenBSD: mdoclint,v 1.60 2017/05/14 13:20:02 schwarze Exp $
 # $NetBSD: mdoclint,v 1.68 2017/05/02 13:59:10 wiz Exp $
 #
 # Copyright (c) 2001-2017 Thomas Klausner
@@ -45,18 +45,18 @@ use constant {
 
 use vars qw(
 	$opt_a $opt_D $opt_d $opt_e $opt_F $opt_f $opt_H $opt_h $opt_l
-	$opt_m $opt_n $opt_o $opt_p $opt_r $opt_S $opt_s $opt_v $opt_w
+	$opt_m $opt_n $opt_o $opt_p $opt_r $opt_S $opt_v $opt_w
 	$opt_X $opt_x
 );
 
 
 my $arch=`uname -m`;
 chomp($arch);
-my $options="aDdeFfHhlmnoprSsvwXx";
+my $options="aDdeFfHhlmnoprSvwXx";
 
 sub usage
 	{
-	my $default = OPENBSD ? "-aDdfmnoprSsXx" : "-aDdeflmnoprSsXx";
+	my $default = OPENBSD ? "-aDdfmnoprSXx" : "-aDdeflmnoprSXx";
 
 	print STDERR <<"EOF";
 mdoclint: verify man page correctness
@@ -76,7 +76,6 @@ usage:	mdoclint [-$options] file ...
 	-p	warn about punctuation problems
 	-r	warn about missing RCS Id
 	-S	warn about any .Sh weirdness
-	-s	warn about whitespace problems
 	-v	verbose output
 	-w	show section header in warnings
 	-X	warn about explicit mentions of the *BSD system names
@@ -254,10 +253,10 @@ sub handle_options
 	unless ($opt_a or $opt_D or $opt_d or $opt_e
 		or $opt_f or $opt_H or $opt_l
 		or $opt_m or $opt_n or $opt_o or $opt_p or $opt_r
-		or $opt_S or $opt_s or $opt_X or $opt_x) {
+		or $opt_S or $opt_X or $opt_x) {
 		$opt_a = $opt_D = $opt_d = $opt_f = $opt_m =
 		    $opt_n = $opt_o = $opt_p = $opt_r = $opt_S =
-		    $opt_s = $opt_X = $opt_x = 1;
+		    $opt_X = $opt_x = 1;
 		$opt_e = $opt_l = 1 if NETBSD;
 	}
 }
@@ -423,10 +422,7 @@ sub process_line
 	($s, $_) = @_;
 	chomp;
 	# always cut trailing spaces
-	if (/\s+$/o) {
-		$s->warning("trailing space: `$_'") if $opt_s;
-		s/\s+$//o;
-	}
+	s/\s+$//o;
 	if (/\$OpenBSD\b.*\$/o) {
 		$s->{oxrcsidseen}++;
 		if (OPENBSD and ($s->{oxrcsidseen} > 1)) {
diff --git a/regress/usr.bin/mdoclint/mdoclint.1 b/regress/usr.bin/mdoclint/mdoclint.1
index cca76a2c680..3b630e46ba5 100644
--- a/regress/usr.bin/mdoclint/mdoclint.1
+++ b/regress/usr.bin/mdoclint/mdoclint.1
@@ -1,4 +1,4 @@
-.\" $OpenBSD: mdoclint.1,v 1.18 2017/05/02 17:51:32 schwarze Exp $
+.\" $OpenBSD: mdoclint.1,v 1.19 2017/05/14 13:20:02 schwarze Exp $
 .\" $NetBSD: mdoclint.1,v 1.15 2017/05/02 13:59:10 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: May 2 2017 $
+.Dd $Mdocdate: May 14 2017 $
 .Dt MDOCLINT 1
 .Os
 .Sh NAME
@@ -33,7 +33,7 @@
 .Nd man page verifier
 .Sh SYNOPSIS
 .Nm
-.Op Fl aDdeFfHhlmnoprSsvwXx
+.Op Fl aDdeFfHhlmnoprSvwXx
 .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 aDdfmnoprSsXx
+.Fl aDdfmnoprSXx
 is assumed (that is, everything except
 .Fl eFHhlvw ) .
 .Pp
@@ -60,8 +60,7 @@ Warn about bad date strings (in the .Dd macro).
 .It Fl e
 Warn about unsorted errors (for functions).
 .It Fl F
-Fix whitespace problems (see also
-.Fl s ) .
+Delete whitespace at the end of input lines.
 .It Fl f
 Warn about possible .Fn abuse; its arguments should be put in
 .Sq \&"
@@ -106,8 +105,6 @@ Warn about missing RCS Id.
 Warn about any unknown sections or about a section that comes in the
 wrong order (see
 .Xr mdoc 7 ) .
-.It Fl s
-Warn about superfluous whitespace at the end of line.
 .It Fl v
 Verbose output.
 .It Fl w