Re: weird bit in _man

[Oliver Kiddle <okiddle@xxxxxxxxxxx>]

Mikael Magnusson wrote:
>     _files || return 0

> surely that should be && ? Or maybe even _files; return since it

I suspect that it is a mistake. The original patch doesn't provide much
clue. && was probably intended but as you say ; is better because you
wouldn't want to go on to complete man pages.  It could perhaps specify
a glob much as _mere does (although allowing for .gz extensions).

The problem with that as a fix is that on some systems man has a -l option
that is something other than view a local file. Or at least, on Solaris,
it prints all the matching manual pages with their section numbers and
location. Perhaps we need to use _pick_variant but I notice that man is
completely different on Debian and on OEL (Oracle's Redhat clone) so it
may not be trivial.

Oliver