[libvirt] [PATCH] esx: Mark error message in macros for translation
Matthias Bolte
matthias.bolte at googlemail.com
Thu Apr 14 15:36:50 UTC 2011
2011/4/11 Eric Blake <eblake at 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
More information about the libvir-list
mailing list