From 7dacdcbd090252d2df7839d7ef04f24fc0fa2b64 Mon Sep 17 00:00:00 2001 From: Vadim Yanitskiy Date: Sat, 15 Dec 2018 20:56:02 +0700 Subject: [PATCH] trx_toolkit/burst_send.py: fix: use proper argparse group Found using Flake8, F841 "local variable 'pf_group' is assigned to but never used". The filtering related options should be defined in 'pf_group' group, not in 'cnt_group'. Change-Id: I15d17c134cbbbd54d761113a56c1f83910ab6407 --- src/target/trx_toolkit/burst_send.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/target/trx_toolkit/burst_send.py b/src/target/trx_toolkit/burst_send.py index 0cf63bb99..7186863bb 100755 --- a/src/target/trx_toolkit/burst_send.py +++ b/src/target/trx_toolkit/burst_send.py @@ -134,13 +134,13 @@ class Application(ApplicationBase): help = "Stop after sending N messages") pf_group = parser.add_argument_group("Filtering (optional)") - cnt_group.add_argument("--timeslot", metavar = "TN", + pf_group.add_argument("--timeslot", metavar = "TN", dest = "pf_tn", type = int, choices = range(0, 8), help = "TDMA timeslot number (equal TN)") - cnt_group.add_argument("--frame-num-lt", metavar = "FN", + pf_group.add_argument("--frame-num-lt", metavar = "FN", dest = "pf_fn_lt", type = int, help = "TDMA frame number (lower than FN)") - cnt_group.add_argument("--frame-num-gt", metavar = "FN", + pf_group.add_argument("--frame-num-gt", metavar = "FN", dest = "pf_fn_gt", type = int, help = "TDMA frame number (greater than FN)")