commit 70ac60b0c03e98d994437d75c0e6f7af7ea1d619
author Brad Bishop <bradleyb@fuzziesquirrel.com> Mon Oct 04 18:48:43 2021 -0400
committer ManojKiran Eda <manojkiran.eda@gmail.com> Wed Oct 06 16:36:10 2021 +0000
tree efae35bac24fb7db864c8be9b8b1a0d00254c837
parent a2b964f281c54457288c28d9f1ed9c1fbb0150c0

pldm_visualise_pdrs.py: remove impossible branch

These arguments are marked as required.  Given that, this branch can
never evaluate to false, so just remove it.

Change-Id: I403cf561b121b28e3fe161f810fc6c8ba26317ee
Signed-off-by: Brad Bishop <bradleyb@fuzziesquirrel.com>

tools/visualize-pdr/pldm_visualise_pdrs.py

diff --git a/tools/visualize-pdr/pldm_visualise_pdrs.py b/tools/visualize-pdr/pldm_visualise_pdrs.py
index 9672361..1a9d087 100755
--- a/tools/visualize-pdr/pldm_visualise_pdrs.py
+++ b/tools/visualize-pdr/pldm_visualise_pdrs.py
@@ -204,12 +204,11 @@
     parser.add_argument('--port', type=int, help="BMC SSH port",
                         default=22)
     args = parser.parse_args()
-    if args.bmc and args.password and args.user:
-        client = connect_to_bmc(args.bmc, args.user, args.password, args.port)
-        association_pdr, state_sensor_pdr, state_effecter_pdr, counter = \
-            fetch_pdrs_from_bmc(client)
-        draw_entity_associations(association_pdr, counter)
-        prepare_summary_report(state_sensor_pdr, state_effecter_pdr)
+    client = connect_to_bmc(args.bmc, args.user, args.password, args.port)
+    association_pdr, state_sensor_pdr, state_effecter_pdr, counter = \
+        fetch_pdrs_from_bmc(client)
+    draw_entity_associations(association_pdr, counter)
+    prepare_summary_report(state_sensor_pdr, state_effecter_pdr)
 
 
 if __name__ == "__main__":