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.