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

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.