Re: [igt-dev] [PATCH i-g-t 5/7] igt/params: add generic saving module parameter set - Juha-Pekka Heikkila

From: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com>
To: Arkadiusz Hiler <arkadiusz.hiler@intel.com>
Cc: igt-dev@lists.freedesktop.org, Jani Nikula <jani.nikula@intel.com>
Subject: Re: [igt-dev] [PATCH i-g-t 5/7] igt/params: add generic saving module parameter set
Date: Tue, 5 May 2020 23:00:03 +0300	[thread overview]
Message-ID: <0169ef69-3a71-991e-0b0c-302b19ffd6ad@gmail.com> (raw)
In-Reply-To: <a9c3b9e6-1f2c-6e08-7c61-74f4aeead861@gmail.com>

On 5.5.2020 21.46, Juha-Pekka Heikkila wrote:
> On 5.5.2020 17.09, Arkadiusz Hiler wrote:
>> On Tue, Apr 28, 2020 at 11:22:53PM +0300, Juha-Pekka Heikkila wrote:
>>> From: Jani Nikula <jani.nikula@intel.com>
>>>
>>> Signed-off-by: Jani Nikula <jani.nikula@intel.com>
>>> Signed-off-by: Juha-Pekka Heikkila <juhapekka.heikkila@gmail.com>
>>> ---
>>>   lib/igt_params.c | 57 ++++++++++++++++++++++++++++++++++++++++--------
>>>   lib/igt_params.h |  3 +++
>>>   2 files changed, 51 insertions(+), 9 deletions(-)
>>>
>>> diff --git a/lib/igt_params.c b/lib/igt_params.c
>>> index fe4b1df3..d9cf986c 100644
>>> --- a/lib/igt_params.c
>>> +++ b/lib/igt_params.c
>>> @@ -153,6 +153,29 @@ int igt_params_open(int device)
>>>       return __igt_params_open(device, NULL);
>>>   }
>>> +__attribute__((format(printf, 3, 0)))
>>
>> this is using a va_list, not "...", so this format attirbue should not
>> be here
>>
> 
> Good catch
> 

Looking it with bit more thought, what's wrong with it? I think it is 
required there.

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev