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 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. :)

I mean it is up to you but with this you might break scripts that are already out there and I think the current commands are quite fine. Cutting out the underscores makes some of these hard to read imho.

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