[libvirt] [PATCH 06/14] Rename suspend capabilities APIs

[Eric Blake]

On 11/29/2011 09:19 AM, Srivatsa S. Bhat wrote:
>>  /**
>> - * virGetPMCapabilities:
>> + * virNodeSuspendGetTargetMask:
>>   *
>>   * Get the Power Management Capabilities that the host system supports,
>>   * such as Suspend-to-RAM (S3), Suspend-to-Disk (S4) and Hybrid-Suspend
>>   * (a combination of S3 and S4).
>>   *
> 
> 
> You might want to remove the terminology "S3", "S4" etc from here, perhaps?

Nah.  While we don't want the public XML or enum names to reflect an
x86-specific naming convention, there's nothing wrong with internal APIs
using terminology in comments to make it easier to correlate the generic
name with the specific feature, so that developers familiar with either
term can be sure to understand.

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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 620 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20111129/6062d131/attachment-0001.sig>