[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240210170055.30fcea14@hermes.local>
Date: Sat, 10 Feb 2024 17:00:55 -0800
From: Stephen Hemminger <stephen@...workplumber.org>
To: Takanori Hirano <me@...tknr.net>
Cc: netdev@...r.kernel.org
Subject: Re: [PATCH iproute2 v2] tc: Add support json option in filter.
On Sat, 10 Feb 2024 10:08:03 +0000
Takanori Hirano <me@...tknr.net> wrote:
>  	}
> +	if (tb[TCA_FLOW_MODE]) {
> +		close_json_object();
> +	}
>  	return 0;
>  }
This last bit is problematic, the JSON encoding should not change
based on whether flow mode is present or not.
Also, brackets not needed around single statement
Powered by blists - more mailing lists
 
