Thread (1 message) 1 message, 1 author, 2012-10-29

Re: [PATCH 03/16] batman-adv: separate BATADV_ATTR_HIF_UINT sysfs name from hard-iface variable

From: Antonio Quartulli <hidden>
Date: 2012-10-29 08:13:40
Also in: batman

Hello David,

On Sun, Oct 28, 2012 at 06:59:27PM -0400, David Miller wrote:
From: Antonio Quartulli <redacted>
Date: Sun, 28 Oct 2012 12:17:04 +0100
quoted
From: Marek Lindner <redacted>

Signed-off-by: Marek Lindner <redacted>
Signed-off-by: Antonio Quartulli <redacted>
---
 net/batman-adv/sysfs.c | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)
I was wondering how you could possibly change the list of arguments to
these macros, yet avoid changing any of the users.

And sure enough it's because these macros are completely unused.
when this macro was introduced a user was going to be added as well, but then we
delayed some work and the user was not introduced anymore..
Please just remove them instead of changing their interfaces when no
users exist.
Yes, I will resend the pull request with a patch that removes this macro at all.
We will re-introduce it as soon as the user is ready too.


Cheers,

-- 
Antonio Quartulli

..each of us alone is worth nothing..
Ernesto "Che" Guevara

Attachments

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help