2011/4/11 Eric Blake <ebl...@redhat.com>:
> On 04/10/2011 05:28 AM, Matthias Bolte wrote:
>> ---
>> src/esx/esx_vi_types.c | 20 ++++++++++----------
>> 1 files changed, 10 insertions(+), 10 deletions(-)
>>
>> diff --git a/src/esx/esx_vi_types.c b/src/esx/esx_vi_types.c
>> index 9e23030..29a3c39 100644
>> --- a/src/esx/esx_vi_types.c
>> +++ b/src/esx/esx_vi_types.c
>> @@ -77,12 +77,12 @@
>> int
>> \
>> esxVI_##__type##_Validate(esxVI_##__type *item)
>> \
>> {
>> \
>> - const char *type_name = esxVI_Type_ToString(esxVI_Type_##__type);
>> \
>> + const char *typeName = esxVI_Type_ToString(esxVI_Type_##__type);
>> \
>>
>> \
>> if (item->_type <= esxVI_Type_Undefined ||
>> \
>> item->_type >= esxVI_Type_Other) {
>> \
>> ESX_VI_ERROR(VIR_ERR_INTERNAL_ERROR,
>> \
>> - "%s object has invalid dynamic type", type_name);
>> \
>> + _("%s object has invalid dynamic type"),
>> typeName); \
>
> Hmm, cfg.mk already included ESX_VI_ERROR in the list of
> msg_gen_function that should have caught this at 'make syntax-check'
> time; I wonder why it was being overlooked. Oh, probably because
> sc_libvirt_unmarked_diagnostics states that it won't find lines where
> the format string is on a separate line. Same goes for
> sc_unmarked_diagnostics from gnulib's maint.mk; wonder if there's a way
> to improve the syntax checker.
>
> At any rate, ACK.
>
Thanks, pushed.
Matthias
--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
