[chirp_devel] [PATCH 03/10] chirpc: add memory option group (#2343)

[Zachary T Welch]

# HG changeset patch
# User Zachary T Welch <zach at mandolincreekfarm.com>
# Fake Node ID 5ca4e798157e01c96a205c719eac80d048909a8e

chirpc: add memory option group (#2343)

This patch groups the memory/channel modification options into an
argument group, making the --help text easier to digest.  It also
eliminates redundant arguments to the relevant add_argument calls,
since those lines are being touched anyway.

diff --git a/chirpc b/chirpc
index 4414fc0..3ca9811 100755
--- a/chirpc
+++ b/chirpc
@@ -77,71 +77,52 @@ if __name__ == "__main__":
                         default=False,
                         action="store_true",
                         help="Request radio ID string")
-    parser.add_argument("--raw", dest="raw",
-                        default=False,
-                        action="store_true",
+
+    memarg = parser.add_argument_group("Memory/Channel Options")
+    memarg.add_argument("--raw", action="store_true",
                         help="Dump raw memory location")
 
-    parser.add_argument("--get-mem", dest="get_mem",
-                        default=False,
-                        action="store_true",
+    memarg.add_argument("--get-mem", action="store_true",
                         help="Get and print memory location")
-    parser.add_argument("--set-mem-name", dest="set_mem_name",
-                        default=None,
-                        help="Set memory name")
-    parser.add_argument("--set-mem-freq", dest="set_mem_freq",
-                        type=float,
-                        default=None,
+    memarg.add_argument("--set-mem-name", help="Set memory name")
+    memarg.add_argument("--set-mem-freq", type=float,
                         help="Set memory frequency")
 
-    parser.add_argument("--set-mem-tencon", dest="set_mem_tencon",
-                        default=False,
-                        action="store_true",
+    memarg.add_argument("--set-mem-tencon", action="store_true",
                         help="Set tone encode enabled flag")
-    parser.add_argument("--set-mem-tencoff", dest="set_mem_tencoff",
-                        default=False,
-                        action="store_true",
+    memarg.add_argument("--set-mem-tencoff", action="store_true",
                         help="Set tone decode disabled flag")
-    parser.add_argument("--set-mem-tsqlon", dest="set_mem_tsqlon",
-                        default=False,
-                        action="store_true",
+    memarg.add_argument("--set-mem-tsqlon", action="store_true",
                         help="Set tone squelch enabled flag")
-    parser.add_argument("--set-mem-tsqloff", dest="set_mem_tsqloff",
-                        default=False,
-                        action="store_true",
+    memarg.add_argument("--set-mem-tsqloff", action="store_true",
                         help="Set tone squelch disabled flag")
-    parser.add_argument("--set-mem-dtcson", dest="set_mem_dtcson",
-                        default=False,
-                        action="store_true",
+    memarg.add_argument("--set-mem-dtcson", action="store_true",
                         help="Set DTCS enabled flag")
-    parser.add_argument("--set-mem-dtcsoff", dest="set_mem_dtcsoff",
-                        default=False,
-                        action="store_true",
+    memarg.add_argument("--set-mem-dtcsoff", action="store_true",
                         help="Set DTCS disabled flag")
 
-    parser.add_argument("--set-mem-tenc", dest="set_mem_tenc",
+    memarg.add_argument("--set-mem-tenc",
                         type=float, action=ToneAction, nargs=1,
                         help="Set memory encode tone")
-    parser.add_argument("--set-mem-tsql", dest="set_mem_tsql",
+    memarg.add_argument("--set-mem-tsql",
                         type=float, action=ToneAction, nargs=1,
                         help="Set memory squelch tone")
 
-    parser.add_argument("--set-mem-dtcs", dest="set_mem_dtcs",
+    memarg.add_argument("--set-mem-dtcs",
                         type=int, action=DTCSAction, nargs=1,
                         help="Set memory DTCS code")
-    parser.add_argument("--set-mem-dtcspol", dest="set_mem_dtcspol",
+    memarg.add_argument("--set-mem-dtcspol",
                         action=DTCSPolarityAction, nargs=1,
                         help="Set memory DTCS polarity (NN, NR, RN, RR)")
 
-    parser.add_argument("--set-mem-dup", dest="set_mem_dup",
+    memarg.add_argument("--set-mem-dup",
                         help="Set memory duplex (+,-, or blank)")
-    parser.add_argument("--set-mem-offset", dest="set_mem_offset",
-                        type=float,
+    memarg.add_argument("--set-mem-offset", type=float,
                         help="Set memory duplex offset (in MHz)")
 
-    parser.add_argument("--set-mem-mode", dest="set_mem_mode",
-                        default=None,
+    memarg.add_argument("--set-mem-mode",
                         help="Set mode (%s)" % ",".join(chirp_common.MODES))
+
     parser.add_argument("-r", "--radio", dest="radio",
                         default=None,
                         help="Radio model (see --list-radios)")