Re: [PATCH] Consistently use 'CANCELED' spelling in public API

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

 



On Mon, Aug 19, 2013 at 11:44:39AM +0200, Christophe Fergeau wrote:
> Most constant names in libvirt public API are using the 'CANCELED'
> spelling, except for VIR_DOMAIN_JOB_CANCELLED and
> VIR_ERR_AUTH_CANCELLED.
> 
> This commit changes the spelling used by these 2 symbols to make
> it consistant with the rest of the API. For backwards compatibility,
> 2 #define are introduced using the old spelling (with 2 'L').
> 
> Signed-off-by: Christophe Fergeau <cfergeau@xxxxxxxxxx>
> ---
>  include/libvirt/libvirt.h.in | 10 +++++++++-
>  include/libvirt/virterror.h  | 10 +++++++++-
>  2 files changed, 18 insertions(+), 2 deletions(-)

This has been proposed before, but rejected. Although the misspelling
sucks, it doesn't really cause any serious pain to apps. Any app that
used the new spelling  would be needlessly tieing itself to new libvirt.
So IMHO it better to just live with this.

Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list




[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]