GNU bug report logs - #12109
23.4; woman.el recognise ".if v"

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 12109 in the body.
You can then email your comments to 12109 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Kevin Ryde <user42 <at> zip.com.au>
To: bug-gnu-emacs <at> gnu.org
Subject: 23.4; woman.el recognise ".if v"
Date: Wed, 01 Aug 2012 10:25:57 +1000

[Message part 1 (text/plain, inline)]

Processing the file ifv.1 below with

    (woman-find-file "ifv.1")

puts a warning in *WoMan-Log*

    **  if v -- unrecognized condition name rejected!

where I thought it might quietly treat ".if v" as false.

According to the groff manual "v" is "-Tversatec" output, whatever or
wherever that is.  Perl pod2man generates it for some accenting with a
comment that it's "for vroff".  Perhaps woman could treat it as false
the same as groff

2012-07-31  Kevin Ryde  <user42 <at> zip.com.au>

	* woman.el (woman0-if): Recognise ".if v" per groff, default false.

[ifv.1 (text/plain, inline)]

.TH FOO 1
.SH NAME
.if v .B something

[woman.el.ifv.diff (text/x-diff, inline)]

--- woman-24.el	2012-07-09 12:07:42.000000000 +1000
+++ woman.el	2012-07-31 10:17:22.000000000 +1000
@@ -2550,7 +2550,8 @@
     (cond
      ;; ((looking-at "[no]") (setq c t))     ; accept n(roff) and o(dd page)
      ;; ((looking-at "[te]") (setq c nil))   ; reject t(roff) and e(ven page)
-     ((looking-at "[ntoe]")
+     ;; Per groff ".if v" is recognised but false (it means -Tversatec).
+     ((looking-at "[ntoev]")
       (setq c (memq (following-char) woman-if-conditions-true)))
      ;; Unrecognized letter so reject:
      ((looking-at "[A-Za-z]") (setq c nil)

[Message part 4 (text/plain, inline)]

In GNU Emacs 23.4.1 (i486-pc-linux-gnu, GTK+ Version 2.24.10)
 of 2012-04-08 on biber, modified by Debian
configured using `configure  '--build' 'i486-linux-gnu' '--build' 'i486-linux-gnu' '--prefix=/usr' '--sharedstatedir=/var/lib' '--libexecdir=/usr/lib' '--localstatedir=/var/lib' '--infodir=/usr/share/info' '--mandir=/usr/share/man' '--with-pop=yes' '--enable-locallisppath=/etc/emacs23:/etc/emacs:/usr/local/share/emacs/23.4/site-lisp:/usr/local/share/emacs/site-lisp:/usr/share/emacs/23.4/site-lisp:/usr/share/emacs/site-lisp' '--with-crt-dir=/usr/lib/i386-linux-gnu' '--with-x=yes' '--with-x-toolkit=gtk' '--with-toolkit-scroll-bars' 'build_alias=i486-linux-gnu' 'CFLAGS=-g -O2 -fstack-protector --param=ssp-buffer-size=4 -Wformat -Wformat-security -Werror=format-security -Wall -DDEBIAN -O2' 'CPPFLAGS=-D_FORTIFY_SOURCE=2''

Important settings:
  value of $LC_ALL: nil
  value of $LC_COLLATE: nil
  value of $LC_CTYPE: nil
  value of $LC_MESSAGES: nil
  value of $LC_MONETARY: nil
  value of $LC_NUMERIC: nil
  value of $LC_TIME: nil
  value of $LANG: en_AU
  value of $XMODIFIERS: nil
  locale-coding-system: iso-latin-1-unix
  default enable-multibyte-characters: t

Message #8 received at 12109 <at> debbugs.gnu.org (full text, mbox):

From: Chong Yidong <cyd <at> gnu.org>
To: Kevin Ryde <user42 <at> zip.com.au>
Cc: 12109 <at> debbugs.gnu.org
Subject: Re: bug#12109: 23.4; woman.el recognise ".if v"
Date: Tue, 07 Aug 2012 11:57:14 +0800

Kevin Ryde <user42 <at> zip.com.au> writes:

> According to the groff manual "v" is "-Tversatec" output, whatever or
> wherever that is.  Perl pod2man generates it for some accenting with a
> comment that it's "for vroff".  Perhaps woman could treat it as false
> the same as groff

Thanks, I've committed your patch to trunk.

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.