Re: bin/52710: Error in vis(1) manpage

[Jamie Landeg-Jones <jamie%dyslexicfish.net@localhost>]

The following reply was made to PR bin/52710; it has been noted by GNATS.

From: Jamie Landeg-Jones <jamie%dyslexicfish.net@localhost>
To: christos%zoulas.com@localhost, tk%giga.or.at@localhost, netbsd-bugs%NetBSD.org@localhost,
        jamie%dyslexicfish.net@localhost, gnats-bugs%NetBSD.org@localhost, gnats-admin%NetBSD.org@localhost
Cc: 
Subject: Re: bin/52710: Error in vis(1) manpage
Date: Wed, 15 Nov 2017 11:44:37 +0000

 Thomas Klausner <tk%giga.or.at@localhost> wrote:
 
 >  I wonder if this is a bug in vis.c instead.
 >  
 >                  case 'F':
 >                          if ((foldwidth = atoi(optarg)) < 5) {
 >                                  errx(1, "can't fold lines to less than 5 cols");
 >                                  /* NOTREACHED */
 >                          }
 >                          markeol++;
 >                          break;
 >                  case 'f':
 >                          fold++;         /* fold output lines to 80 cols */
 >                          break;          /* using hidden newline */
 >  
 >  Perhaps this should be "fold++" instead of "markeol++".
 >   Thomas
 
 I see what you're saying: Rather than '-F' simply setting the columns for use by '-f', the
 '-F' would basically imply the '-f', and be used instead of it.
 
 Hmm, make sense.. The manpage would still be wrong, but not quite as much!
 
 Cheers, Jamie