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

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

 



On Tue, Apr 05, 2022 at 01:43:23PM +0200, Victor Toso wrote:
> Differently from the previous patches, we don't parse nor export
> comments associated with variables. This isn't a big deal because we
> only export a single variable: virConnectAuthPtrDefault
>
> Nonetheless, add version field to the exported XML by checking the
> allowlist file. This way, if we add another variable in the future, we
> can simply add it to that file. Calling the function should also warn
> in case we are exporting a new Variable without adding to the file,
> e.g:
>     Missing 'Since' tag for: virConnectAuthPtrDefault

I think the fact that we're currently not parsing comments for
variables is just a mistake.

virConnectAuthPtrDefault seems to be documented in a way that would
be appropriate for the API docs, and the fact that it doesn't show up
there means that users of the library have no way to figure out what
it's there for without digging into the source code, or even that it
exists at all.

I'd say just start treating comments for variables the same as those
for all other symbols.

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