[PATCH v1 2/4] scripts: apibuild: parse 'Since' for typedefs
Victor Toso
victortoso at redhat.com
Tue Apr 5 15:36:51 UTC 2022
Hi,
On Tue, Apr 05, 2022 at 04:52:09PM +0200, Michal Prívozník wrote:
> On 4/5/22 13:43, Victor Toso wrote:
> > This patch adds 'version' parameter to the generated XML API for
> > typedefs
> >
> > It'll require, for new additions, to add a comment with the version
> > that the typedef value was added. An example bellow of code diff and
> > the change in the generated XML.
> >
> > Note that the Since tag is removed from the comment as there is a
> > proper field for it in the XML.
> >
> > ```diff
> > --- a/include/libvirt/libvirt-host.h
> > +++ b/include/libvirt/libvirt-host.h
> > @@ -41,6 +41,8 @@ typedef struct _virConnect virConnect;
> > *
> > * a virConnectPtr is pointer to a virConnect private structure, this is the
> > * type used to reference a connection to the Hypervisor in the API.
> > + *
> > + * Since 0.0.1
> > */
> > typedef virConnect *virConnectPtr;
> > ```
> >
> > ```xml
> > <typedef name='virConnectPtr'
> > file='libvirt-host'
> > type='virConnect *'
> > version='0.0.1'>
> > <info><![CDATA[a virConnectPtr is pointer to a virConnect private
> > structure, this is the type used to reference a connection to the
> > Hypervisor in the API.]]></info>
> > </typedef>
> > ```
> >
>
> I'm not exactly sure why, but this diff in commit message causes my git
> to think this is a broken patch. Maybe it's trying to parse it as an
> actual patch? Funnily enough, there's no problem with 1/4.
>
> Michal
When I send a v2, I'll make both examples above quoted, that is,
prefixed with: " > ". Not sure why I actually used ```diff and
```xml, gitlab does nothing with it x)
Cheers,
Victor
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20220405/681cf4bb/attachment.sig>
More information about the libvir-list
mailing list