[LTP] [PATCH v4] save_restore: Introduce new struct field for flags

Martin Doucha mdoucha@suse.cz
Wed Nov 16 16:51:15 CET 2022


On 16. 11. 22 16:46, Cyril Hrubis wrote:
>> +static void print_error(int info_only, const char *err, const char *path)
>> +{
>> +	if (info_only)
>> +		tst_res(TINFO | TERRNO, err, path);
>> +	else
>> +		tst_brk(TBROK | TERRNO, err, path);
>> +}
> 
> If I remmeber correctly I did ask for this to become a macro or pass
> __LINE__ so that we have a correct lines in the error output and you
> haven't commented on that.
> 
> Otherwise the rest looks good to me.

Right, sorry, I forgot about the line numbers. I'll send v5 shortly.

-- 
Martin Doucha   mdoucha@suse.cz
QA Engineer for Software Maintenance
SUSE LINUX, s.r.o.
CORSO IIa
Krizikova 148/34
186 00 Prague 8
Czech Republic



More information about the ltp mailing list