[FFmpeg-devel] [PATCH 1/7] checkasm: add csv/tsv bench output

[J. Dekker]

On Fri, Aug 16, 2024, at 13:03, Anton Khirnov wrote:
> Quoting J. Dekker (2024-08-13 16:03:30)
>> When collecting performance information from checkasm it is common
>> to parse the output for use in graphs to compare vs different
>> architectures.
>> 
>> Signed-off-by: J. Dekker <jdek at itanimul.li>
>> ---
>>  tests/checkasm/checkasm.c | 22 ++++++++++++++++++++--
>>  1 file changed, 20 insertions(+), 2 deletions(-)
>
> Is TSV a standard acronym? I've never seen it before.
> And would it make sense to allow specifying an arbitrary separator
> string?

I think TSV is understood within the context of CSV. Initially I was going to just have only the ‘standard’ CSV but I would personally prefer a whitespace separator like <TAB> (I have seen the ASCII control characters used too). I don’t think that it would make sense to support any further arbitrary characters since you can s/\t/<sep>/g.

jd