Re: [PATCH v3 13/30] docstring: function: libvirt: Add 'Since version' metadata

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

 



On Fri, Apr 22, 2022 at 10:44:05AM +0200, Victor Toso wrote:
> On Fri, Apr 22, 2022 at 01:29:07AM -0700, Andrea Bolognani wrote:
> > On Thu, Apr 21, 2022 at 09:12:10PM +0200, Victor Toso wrote:
> > > Yeah. I'd instead suggest to move documentation to the top of
> > > typedef enum, in a similar fashion of what qemu/qapi schema does.
> > >
> > >     https://github.com/qemu/qemu/blob/master/qapi/audio.json#L13
> > >
> > > Documentation is kept close enough of the data type without
> > > polluting the surround definitions, etc. At the very least, it
> > > would use less empty lines... I think.
> >
> > Oh, I hadn't even considered that as a possibility but it does indeed
> > look promising. We'd have to see how it look in practice to actually
> > judge though. Maybe you can prototype that after we're done with this
> > series, if you feel like it? :)
>
> Sure, I can't promise it'll be soon after this series is done,
> but it'll be on my list.
>
> What is very important is to have a well defined structure for
> the docstrings as we'll need to do extra work on apibuild.py.
>
> Also, would make sense to add other kind of metadata besides the
> Since version?

I don't know :) Let's put this idea on hold for now and focus on
getting the version information merged.

-- 
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