Re: [PATCH wpan-tools 1/2] treewide: shorten commands

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello.

On 20/01/15 12:32, Alexander Aring wrote:
On Tue, Jan 20, 2015 at 11:05:26AM +0000, Stefan Schmidt wrote:
Hello.

On 20/01/15 11:35, Alexander Aring wrote:
This patch shorten all commands. I had the idea to make a similarity
>from PIB/MIB values according the 802.15.4 names convention but this is
bad for fast shell hacking. Mainly I remove the underscore and some
additional information which are not necessary like "_addr" in
"short_addr" and "extended_addr" and such things.

Signed-off-by: Alexander Aring <alex.aring@xxxxxxxxx>
---
  src/info.c      | 10 +++++-----
  src/interface.c | 20 ++++++++++----------
  src/mac.c       | 10 +++++-----
  src/phy.c       |  6 +++---
  4 files changed, 23 insertions(+), 23 deletions(-)

Hmm, it feels like you are going overboard with shorting things here. Why
is
the reason behind it? Just less typing? Shell auto completion would handle
this. :)


it's just less typing and I notice that the original iw command doesn't
have an underscore in their commands.

The first time when I hacked it, then I just took the same naming
convention in kernel implementation which use the naming convention from
802.15.4 standard (without the beginning "MAC" and "PHY" keywords).

Having the naming the same in specs, kernel and the config util is actually good thing. This helps newcomers to understand them more easily.

Now I am too lazy to always type pan_id short_addr, etc...

If the primary goal would be less typing the the binary should be named "i" and the options a, b, c, d :)

Do you like to implement some completion stuff which can be shipped out
optionally with the wpan-tools? :-)

Here is a start:
_iwpan()
{
    local cur prev opts
    COMPREPLY=()
    cur="${COMP_WORDS[COMP_CWORD]}"
    prev="${COMP_WORDS[COMP_CWORD-1]}"
opts="--debug --version phy dev set channel tx_power cca_mode cca_ed_level pan_id short_addr max_frame_retries backoff_exponents max_csma_backoffs lbt info list interface add"

    COMPREPLY=( $(compgen -W "${opts}" -- ${cur}) )
    return 0
}
complete -F _iwpan iwpan

Just drop this in a file called iwpan in /etc/bash_completion.d/ and a simple command completion would work with bash. zsh user might want to do their own.

This has no fancy things like completing available wpan interfaces, etc but it completes all commands which was what you complained about for typing. :)

regards
Stefan Schmidt


--
To unsubscribe from this list: send the line "unsubscribe linux-wpan" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux