Thread (6 messages) flat view 6 messages, 2 authors, 2021-08-14

Re: [PATCH iproute2-next 2/3] bridge: fdb: don't colorize the "dev" & "dst" keywords in "bridge -c fdb"

From: Stephen Hemminger <stephen@networkplumber.org>
Date: 2021-08-14 16:22:37

On Sat, 14 Aug 2021 15:24:38 +0530
Gokul Sivakumar [off-list ref] wrote:
+	if (!filter_index && r->ndm_ifindex) {
+		if (!is_json_context())
+			fprintf(fp, "dev ");
This looks functionally correct, but please use:
            print_string(PRINT_FP, NULL, "dev ", NULL);

The reason as part of the json conversions I look for fprintf(fp
as indicator of unconverted code.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help