[prev in list] [next in list] [prev in thread] [next in thread] 

List:       busybox
Subject:    Re: Add possiblity to support specifying desired section of manpage
From:       Denys Vlasenko <vda.linux () googlemail ! com>
Date:       2020-12-27 16:45:57
Message-ID: CAK1hOcOeiY_c1moZK-5dc14xw8P6L_i0uUiQNu5Lz7yf5+XDMQ () mail ! gmail ! com
[Download RAW message or body]

miscutils/man.c: In function ‘man_main':
miscutils/man.c:343:2: error: ISO C90 forbids mixed declarations and
code [-Werror=declaration-after-statement]
  343 |  char sect_str[3];
      |  ^~~~
miscutils/man.c:395:4: error: ISO C90 forbids mixed declarations and
code [-Werror=declaration-after-statement]
  395 |    const char *cur_sect = sec_list;
      |    ^~~~~
  CC      libbb/vfork_daemon_rexec.o
miscutils/man.c:390:14: error: ‘sect_len' may be used uninitialized in
this function [-Werror=maybe-uninitialized]
  390 |     found |= check_path(cur_path, sect_str, sect_len, *argv);

I can implement it differently. Please try current git.

On Tue, Dec 22, 2020 at 7:10 PM depsterr <depsterr@protonmail.com> wrote:
> 
> I have created a new patch which handles both the
> 
> section page
> 
> and
> 
> page.section
> 
> formats. (The page(section) format has been ignored as it is less common and more \
> tedious to implement.) 
> This patch also fixes a few typos in the previous patch. This should be my final \
> version of this patch. 
> Thanks to everyone for the feedback! Hopefully this can get merged :)
> 
> ------- Original Message -------
> On Tuesday, December 22, 2020 5:15 PM, Harald van Dijk <harald@gigawatt.nl> wrote:
> 
> > On 22/12/2020 15:42, Xabier Oneca -- xOneca wrote:
> > 
> > > Hi Depsterr,
> > > 
> > > > Usage: man [-aw] MANPAGE[.SECTION]...
> > > 
> > > That does not seem to be compatible with the standard man:
> > > man [OPTION...] [SECTION] PAGE...
> > 
> > It is not so much incompatible as it is incomplete. The widely used
> > man-db implementation of man does support this form as well. It supports
> > three forms of specifying the section, which all do the same thing:
> > 
> > man 7 man
> > man man.7
> > man 'man(7)'
> > 
> > Cheers,
> > Harald van Dijk
> 
> _______________________________________________
> busybox mailing list
> busybox@busybox.net
> http://lists.busybox.net/mailman/listinfo/busybox
_______________________________________________
busybox mailing list
busybox@busybox.net
http://lists.busybox.net/mailman/listinfo/busybox


[prev in list] [next in list] [prev in thread] [next in thread]