* [B.A.T.M.A.N.] [PATCH] alfred: Keep order of interfaces from commandline
@ 2016-10-13 9:03 Sven Eckelmann
2016-10-14 13:33 ` Simon Wunderlich
0 siblings, 1 reply; 2+ messages in thread
From: Sven Eckelmann @ 2016-10-13 9:03 UTC (permalink / raw)
To: b.a.t.m.a.n; +Cc: Sven Eckelmann
The first (valid) interface mac address will be used as source mac for an
dataset. But alfred added the commandline parameters for a multi-interface
master in reverse order.
Keep the list in the same order as the commandline to fulfill the
user expectations when checking the source entries of datasets.
Signed-off-by: Sven Eckelmann <sven.eckelmann@open-mesh.com>
---
netsock.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/netsock.c b/netsock.c
index d72541e..5d37347 100644
--- a/netsock.c
+++ b/netsock.c
@@ -165,7 +165,7 @@ int netsock_set_interfaces(struct globals *globals, char *interfaces)
return -ENOMEM;
}
- list_add(&interface->list, &globals->interfaces);
+ list_add_tail(&interface->list, &globals->interfaces);
}
return 0;
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [B.A.T.M.A.N.] [PATCH] alfred: Keep order of interfaces from commandline
2016-10-13 9:03 [B.A.T.M.A.N.] [PATCH] alfred: Keep order of interfaces from commandline Sven Eckelmann
@ 2016-10-14 13:33 ` Simon Wunderlich
0 siblings, 0 replies; 2+ messages in thread
From: Simon Wunderlich @ 2016-10-14 13:33 UTC (permalink / raw)
To: b.a.t.m.a.n; +Cc: Sven Eckelmann
[-- Attachment #1: Type: text/plain, Size: 500 bytes --]
On Thursday, October 13, 2016 11:03:10 AM CEST Sven Eckelmann wrote:
> The first (valid) interface mac address will be used as source mac for an
> dataset. But alfred added the commandline parameters for a multi-interface
> master in reverse order.
>
> Keep the list in the same order as the commandline to fulfill the
> user expectations when checking the source entries of datasets.
>
> Signed-off-by: Sven Eckelmann <sven.eckelmann@open-mesh.com>
Applied in revision f21f747
Thanks!
Simon
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 801 bytes --]
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2016-10-14 13:33 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-10-13 9:03 [B.A.T.M.A.N.] [PATCH] alfred: Keep order of interfaces from commandline Sven Eckelmann
2016-10-14 13:33 ` Simon Wunderlich
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox