Re: [PATCH v1 4/4] scripts: apibuild: add 'version' to variables

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Apr 06, 2022 at 10:49:47AM +0200, Victor Toso wrote:
> On Wed, Apr 06, 2022 at 01:10:40AM -0700, Andrea Bolognani wrote:
> > On Tue, Apr 05, 2022 at 08:31:40PM +0200, Victor Toso wrote:
> > > The documentation is not in the headers:
> > >
> > >     https://gitlab.com/libvirt/libvirt/-/blob/master/include/libvirt/libvirt-host.h#L565
> > >
> > > Only in the source:
> > >
> > >     https://gitlab.com/libvirt/libvirt/-/blob/master/src/libvirt.c#L200
> > >
> > > So, moving the documentation around could be an extra patch,
> > > indeed.
> >
> > A lot of documentation lives in the .c file, notably that for
> > functions. It's perfectly fine for it to be there and it doesn't need
> > to be moved.
>
> The suggestion of moving was to address what you pointed out
> previously, that users of libvirt can't figure out what that
> variable does without digging into the source code.

Once the docstring for the variable is parsed and it shows up in the
HTML documentation, it becomes just as discoverable as any of the
functions, which is perfectly fine :)

-- 
Andrea Bolognani / Red Hat / Virtualization




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux