[libvirt] [PATCH] Add a test suite for keycode mapping functions

Eric Blake eblake at redhat.com
Fri Apr 5 17:00:05 UTC 2013


On 04/05/2013 10:53 AM, Eric Blake wrote:
> On 04/05/2013 10:50 AM, Daniel P. Berrange wrote:
>> From: "Daniel P. Berrange" <berrange at redhat.com>
>>
>> Validate that translations between different keycode sets
>> are functioning.
>>
>> Signed-off-by: Daniel P. Berrange <berrange at redhat.com>
>> ---
>>  tests/virkeycodetest.c | 107 +++++++++++++++++++++++++++++++++++++++++++++++++
>>  1 file changed, 107 insertions(+)
>>  create mode 100644 tests/virkeycodetest.c
> 
> Never hurts to add tests :)

But you might want to touch up .gitignore so future users don't commit a
binary into .git by accident when doing 'git add -a'.

> 
>> +static int
>> +mymain(void)
>> +{
>> +    int ret = 0;
>> +
>> +    if (virtTestRun("Keycode mapping ", 1, testKeycodeMapping, NULL) < 0)
>> +        ret = -1;
>> +    if (virtTestRun("Keycode strings ", 1, testKeycodeStrings, NULL) < 0)
>> +        ret = -1;
>> +
>> +    return ret==0 ? EXIT_SUCCESS : EXIT_FAILURE;
> 
> Spaces around ==.
> 
> ACK.
> 
> 
> 
> --
> libvir-list mailing list
> libvir-list at redhat.com
> https://www.redhat.com/mailman/listinfo/libvir-list
> 

-- 
Eric Blake   eblake 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: 621 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20130405/375b546f/attachment-0001.sig>


More information about the libvir-list mailing list