Removes an unneeded line from the two scripts; Use context manager in player.py (#1099)

* Removes an unneeded line from the two scripts each

Was discussed in #1072

* Tiny cleanup in plyer.py

Author: felixdivo

can/logger.py

@@ -124,7 +124,6 @@ def _parse_filters(parsed_args: Any) -> CanFilters:
 
 def main() -> None:
     parser = argparse.ArgumentParser(
-        "python -m can.logger",
         description="Log CAN traffic, printing messages to stdout or to a given file.",
     )
 

can/player.py

@@ -13,14 +13,11 @@
 
 from can import LogReader, Message, MessageSync
 
-
 from .logger import _create_base_argument_parser, _create_bus
 
 
 def main() -> None:
-    parser = argparse.ArgumentParser(
-        "python -m can.player", description="Replay CAN traffic."
-    )
+    parser = argparse.ArgumentParser(description="Replay CAN traffic.")
 
     _create_base_argument_parser(parser)
 
@@ -87,31 +84,27 @@ def main() -> None:
 
     error_frames = results.error_frames
 
-    bus = _create_bus(results)
-
-    reader = LogReader(results.infile)
-
-    in_sync = MessageSync(
-        cast(Iterable[Message], reader),
-        timestamps=results.timestamps,
-        gap=results.gap,
-        skip=results.skip,
-    )
-
-    print(f"Can LogReader (Started on {datetime.now()})")
-
-    try:
-        for m in in_sync:
-            if m.is_error_frame and not error_frames:
-                continue
-            if verbosity >= 3:
-                print(m)
-            bus.send(m)
-    except KeyboardInterrupt:
-        pass
-    finally:
-        bus.shutdown()
-        reader.stop()
+    with _create_bus(results) as bus:
+        with LogReader(results.infile) as reader:
+
+            in_sync = MessageSync(
+                cast(Iterable[Message], reader),
+                timestamps=results.timestamps,
+                gap=results.gap,
+                skip=results.skip,
+            )
+
+            print(f"Can LogReader (Started on {datetime.now()})")
+
+            try:
+                for message in in_sync:
+                    if message.is_error_frame and not error_frames:
+                        continue
+                    if verbosity >= 3:
+                        print(message)
+                    bus.send(message)
+            except KeyboardInterrupt:
+                pass
 
 
 if __name__ == "__main__":