From: Christoffer Holmstedt <christoffer@christofferholmstedt.se>
To: Alexander Aring <alex.aring@gmail.com>
Cc: Varka Bhadram <varkabhadram@gmail.com>, linux-wpan@vger.kernel.org
Subject: Re: [PATCH wpan-tools] info: add frequency to channel listing for phy capabilities
Date: Tue, 2 Jun 2015 09:38:27 +0200 [thread overview]
Message-ID: <20150602073827.GA3273@probook-6560b> (raw)
In-Reply-To: <20150602070926.GB1977@omega>
On Tue, Jun 02, 2015 at 09:09:30AM +0200, Alexander Aring wrote:
> On Tue, Jun 02, 2015 at 08:08:38AM +0200, Christoffer Holmstedt wrote:
> ...
> >
> > This is for the same reason as above. Alignment in the output, it is a trade
> > off between "readability" in two different ways. Alignment in the output or
> > printing the full word "Unknown". I will change this to "Unknown" and we can
> > change it at a later date if it ever shows up and mess up alignment. In any
> > case if "Unknown" shows up the actual frequency should be added instead of
> > fixing alignment. ;)
> >
>
> Then maybe some "-" instead "unkown", is this better, or it's too small
> then?
>
The more I think about it the more I feel "Unknown" is better. If for some
reason the channel page and channel combination is not listed and "Unknown" is
printed the last thing the user will care about is alignment in the output.
Channel page and channel number information will not change that often so we
should be able to keep up with future changes (additions). I have tried to get
hold of the 802.15.4-2015 draft from March/April but without success. The local
university will charge me ~350€ or something in that region for it. I wanted it
to see if any new channel pages/channels have been added since -2011 edition.
Anyway, v2 coming in a few minutes.
> - Alex
--
Christoffer Holmstedt
next prev parent reply other threads:[~2015-06-02 7:38 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-06-01 13:35 [PATCH wpan-tools] info: add frequency to channel listing for phy capabilities Christoffer Holmstedt
2015-06-02 4:10 ` Varka Bhadram
2015-06-02 6:08 ` Christoffer Holmstedt
2015-06-02 7:09 ` Alexander Aring
2015-06-02 7:38 ` Christoffer Holmstedt [this message]
2015-06-02 9:34 ` Lennert Buytenhek
2015-06-02 9:53 ` Christoffer Holmstedt
2015-06-02 10:15 ` Alexander Aring
2015-06-02 10:37 ` Lennert Buytenhek
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20150602073827.GA3273@probook-6560b \
--to=christoffer@christofferholmstedt.se \
--cc=alex.aring@gmail.com \
--cc=linux-wpan@vger.kernel.org \
--cc=varkabhadram@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox