On Thu, Apr 21, 2022 at 15:51:25 +0200, Peter Krempa wrote:
> On Wed, Apr 20, 2022 at 21:08:09 +0200, Victor Toso wrote:
> > This commit is similar to "docs: Fix generated documentation of
> > virConnectListAllNodeDeviceFlags", check it out for more info.
> > 
> > Signed-off-by: Victor Toso <victort...@redhat.com>
> > ---
> >  include/libvirt/libvirt-domain.h | 14 +++++++-------
> >  1 file changed, 7 insertions(+), 7 deletions(-)
> > 
> > diff --git a/include/libvirt/libvirt-domain.h 
> > b/include/libvirt/libvirt-domain.h
> > index 9199cc9ad3..27a680a907 100644
> > --- a/include/libvirt/libvirt-domain.h
> > +++ b/include/libvirt/libvirt-domain.h
> > @@ -2758,18 +2758,18 @@ int                     virDomainGetVcpus       
> > (virDomainPtr domain,
> >  /**
> >   * virDomainDeviceModifyFlags:
> >   *
> > - * Since: v1.0.0
> > + * These flags may be bitwise-OR'd in.
> > + *
> > + * Since: v0.7.7
> >   *
> >   */
> >  typedef enum {
> > -    /* See virDomainModificationImpact for these flags. (Since: v1.0.0)  */
> > -    VIR_DOMAIN_DEVICE_MODIFY_CURRENT = VIR_DOMAIN_AFFECT_CURRENT,
> > -    VIR_DOMAIN_DEVICE_MODIFY_LIVE    = VIR_DOMAIN_AFFECT_LIVE,
> > -    VIR_DOMAIN_DEVICE_MODIFY_CONFIG  = VIR_DOMAIN_AFFECT_CONFIG,
> > +    VIR_DOMAIN_DEVICE_MODIFY_CURRENT = VIR_DOMAIN_AFFECT_CURRENT, /* See 
> > virDomainModificationImpact (Since: v0.7.7) */
> > +    VIR_DOMAIN_DEVICE_MODIFY_LIVE    = VIR_DOMAIN_AFFECT_LIVE, /* (Since: 
> > v0.7.7) */
> > +    VIR_DOMAIN_DEVICE_MODIFY_CONFIG  = VIR_DOMAIN_AFFECT_CONFIG, /* 
> > (Since: v0.7.7) */
> >  
> > -    /* Additionally, these flags may be bitwise-OR'd in. (Since: v1.0.0)  
> > */
> >      VIR_DOMAIN_DEVICE_MODIFY_FORCE = (1 << 2), /* Forcibly modify device
> > -                                                  (ex. force eject a 
> > cdrom) */
> > +                                                  (ex. force eject a 
> > cdrom) (Since: v0.8.6) */
> >  } virDomainDeviceModifyFlags;
> 
> This one has a lot of weird 'Since' tags.

I'm unsure how to get about this. Either squash this to the appropriate
commit or perhaps remove the comments breaking stuff before and then
just apply the numbers along in the patch doing everything else.

Reply via email to