2009-02-03 00:10:55

by Pat Erley

Subject: [patch] iw: align output for HT capabilities

The HT capabilities have an additional space in front of them causing
them not to line up with the rest of the listed outputs. This patch
removes that space.

Signed-off-by: Pat Erley <[email protected]>

---

diff --git a/info.c b/info.c
index bf3b8bd..69be500 100644
--- a/info.c
+++ b/info.c
@@ -70,7 +70,7 @@ static int print_phy_handler(struct nl_msg *msg, void
*arg)
#ifdef NL80211_BAND_ATTR_HT_CAPA
if (tb_band[NL80211_BAND_ATTR_HT_CAPA]) {
unsigned short cap = nla_get_u16(tb_band[NL80211_BAND_ATTR_HT_CAPA]);
-#define PCOM(fmt, args...) do { printf("\t\t\t * " fmt "\n", ##args); }
while (0)
+#define PCOM(fmt, args...) do { printf("\t\t\t* " fmt "\n", ##args); }
while (0)
#define PBCOM(bit, args...) if (cap & (bit)) PCOM(args)
printf("\t\tHT capabilities: 0x%.4x\n", cap);
PBCOM(0x0001, "LPDC coding");

2009-02-03 00:15:49

by Julian Calaby

Subject: Re: [patch] iw: align output for HT capabilities

On Tue, Feb 3, 2009 at 11:10, pat-lkml <[email protected]> wrote:
> The HT capabilities have an additional space in front of them causing
> them not to line up with the rest of the listed outputs. This patch
> removes that space.
>
> Signed-off-by: Pat Erley <[email protected]>
>
> ---
>
> diff --git a/info.c b/info.c
> index bf3b8bd..69be500 100644
> --- a/info.c
> +++ b/info.c
> @@ -70,7 +70,7 @@ static int print_phy_handler(struct nl_msg *msg, void
> *arg)
> #ifdef NL80211_BAND_ATTR_HT_CAPA
> if (tb_band[NL80211_BAND_ATTR_HT_CAPA]) {
> unsigned short cap = nla_get_u16(tb_band[NL80211_BAND_ATTR_HT_CAPA]);
> -#define PCOM(fmt, args...) do { printf("\t\t\t * " fmt "\n", ##args); }
> while (0)
> +#define PCOM(fmt, args...) do { printf("\t\t\t* " fmt "\n", ##args); }
> while (0)

Is it just me or is this patch line-wrapped?

Thanks,

--

Julian Calaby

Email: [email protected]
.Plan: http://sites.google.com/site/juliancalaby/

2009-02-03 00:18:54

by Pat Erley

Subject: Re: [patch] iw: align output for HT capabilities

Julian Calaby wrote:
> On Tue, Feb 3, 2009 at 11:10, pat-lkml <[email protected]> wrote:
>> The HT capabilities have an additional space in front of them causing
>> them not to line up with the rest of the listed outputs. This patch
>> removes that space.
>>
>> Signed-off-by: Pat Erley <[email protected]>
>>
>> ---
>>
>> diff --git a/info.c b/info.c
>> index bf3b8bd..69be500 100644
>> --- a/info.c
>> +++ b/info.c
>> @@ -70,7 +70,7 @@ static int print_phy_handler(struct nl_msg *msg, void
>> *arg)
>> #ifdef NL80211_BAND_ATTR_HT_CAPA
>> if (tb_band[NL80211_BAND_ATTR_HT_CAPA]) {
>> unsigned short cap = nla_get_u16(tb_band[NL80211_BAND_ATTR_HT_CAPA]);
>> -#define PCOM(fmt, args...) do { printf("\t\t\t * " fmt "\n", ##args); }
>> while (0)
>> +#define PCOM(fmt, args...) do { printf("\t\t\t* " fmt "\n", ##args); }
>> while (0)
>
> Is it just me or is this patch line-wrapped?
>
> Thanks,
>
no, it line wrapped. Weird that it'd line wrap the function def but not
the line later on me, but I guess it's time to look into fixing my mail
client.

Thanks,
Pat