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

Re: [libvirt] question on API spelling



On 04/03/2012 03:10 AM, Martin Kletzander wrote:
> On 04/03/2012 01:05 AM, Eric Blake wrote:
>> We're quite inconsistent on US (canceled) vs. UK (cancelled) spelling in
>> our public API:
>>
>> include/libvirt/libvirt.h.in:124:
>> VIR_DOMAIN_RUNNING_MIGRATION_CANCELED = 6,  /* returned from migration */
>> include/libvirt/libvirt.h.in:125:    VIR_DOMAIN_RUNNING_SAVE_CANCELED =
>> 7,   /* returned from failed save process */
>> include/libvirt/libvirt.h.in:3220:    VIR_DOMAIN_JOB_CANCELLED = 5, /*
>> Job was aborted, but isn't cleaned up */
>> include/libvirt/virterror.h:247:
>>    canceled/aborted by user */
>> include/libvirt/virterror.h:248:    VIR_ERR_AUTH_CANCELLED = 79,
>> /* authentication cancelled */
>>
>> Is it worth adding aliases so that a programmer can use consistent spelling?
>>
> 
> I think it's worth it, because it looks pretty bad like this.
> I just wonder if it could confuse someone, but that's probably too
> paranoid idea :)

Do either of the Dan's have a strong opinion?  And if the consensus is
indeed for a common spelling, do we favor US, favor UK, or provide
aliases for all of these instances?  History-wise, we've been using
CANCELED since commit d65a924 (May 2011), but CANCELLED in commits
7d575e0 (Mar 2010) and bb2eddc (Jan 2012).  My work on block job will be
introducing another instance (the event for async block job
cancelation), so it would be nice to know which way to spell my patch.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature


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