Subject: Re: [PATCH iproute2-next 2/3] utils: timestamp: add JSON support

From: Stephen Hemminger <[email-masked]>
Date: Fri, 20 Feb 2026 21:39:00 -0800

On Fri, 20 Feb 2026 19:54:02 +0100
"Matthieu Baerts (NGI0)" <[email-masked]> wrote:

> Only if the output stream is 'stdout', because all JSON helpers like
> print_string() only write on 'stdout'.
> 
> Supporting JSON is easy with the helpers. The biggest modification is to
> extract the end value.
> 
> No behavioural changes intended for the moment, this is a preparation for a
> future usage of print_timestamp() within a JSON context.
> 
> Signed-off-by: Matthieu Baerts (NGI0) <[email-masked]>

I would go farther, fp is always stdout. Drop the argument to the function.