[PATCH v1 2/4] scripts: apibuild: parse 'Since' for typedefs

Michal Prívozník mprivozn at redhat.com
Tue Apr 5 14:52:09 UTC 2022


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



More information about the libvir-list mailing list