bug#5686: M-x man thinks the "/O" of "I/O" is a clickable file path

bug#5686: M-x man thinks the "/O" of "I/O" is a clickable file path

[jidanni]

M-x man thinks the "/O" of "I/O" is a clickable file path.
$ man mount|grep I/O|sed q
       async  All I/O to the filesystem should be done asynchronously.
C-u C-x = shows
 There are text properties here:
   Man-target-string    "/O"
   button               (t)
   category             Man-xref-normal-file-button

Anyways relative file paths like bla/zla usually aren't useful either
for clicking on on a man page, so perhaps adjust the regexp involved to
not assume that they are to be made clickable.

In emacs -Q -nw the /O of I/O glows blue, so it is much more noticeable
than the underlined form seen in X windows. I wouldn't have noticed if I
didn't use emacs -nw. emacs-version "23.1.93.1".

bug#5686: M-x man thinks the "/O" of "I/O" is a clickable file path

[Stefan Monnier]

> M-x man thinks the "/O" of "I/O" is a clickable file path.

Thanks.  I've just installed the patch below which should fix it.


        Stefan


--- lisp/man.el	2010-02-16 14:35:45 +0000
+++ lisp/man.el	2010-03-05 20:15:07 +0000
@@ -283,7 +283,7 @@
   "Regular expression for SYNOPSIS heading (or your equivalent).
 This regexp should not start with a `^' character.")
 
-(defvar Man-files-regexp "FILES"
+(defvar Man-files-regexp "FILES\\>"
   "Regular expression for FILES heading (or your equivalent).
 This regexp should not start with a `^' character.")
 

bug#5686: M-x man thinks the "/O" of "I/O" is a clickable file path

[Stefan Monnier]

>> M-x man thinks the "/O" of "I/O" is a clickable file path.
> Thanks.  I've just installed the patch below which should fix it.

Forgot to close it, sorry,


        Stefan