Re: [PATCH v3 20/30] docstring: Fix and append Since to virDomainDeviceModifyFlags

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

 



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 <victortoso@xxxxxxxxxx>
> > ---
> >  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.




[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