From 3dfeee9332f4436467056178d7ddc3ff3a2aa8a8 Mon Sep 17 00:00:00 2001 From: Trenton H <797416+stumpylog@users.noreply.github.com> Date: Fri, 11 Nov 2022 10:09:56 -0800 Subject: [PATCH] Don't do decoding work if not needed --- src/documents/consumer.py | 32 ++++++++++++++++---------------- 1 file changed, 16 insertions(+), 16 deletions(-) diff --git a/src/documents/consumer.py b/src/documents/consumer.py index 45722d01a..75181cabb 100644 --- a/src/documents/consumer.py +++ b/src/documents/consumer.py @@ -537,27 +537,27 @@ class Consumer(LoggingMixin): ) # Decode the output (if any) - stdout_str = ( - completed_process.stdout.decode("utf8", errors="ignore") - .strip() - .split( - "\n", + if len(completed_process.stdout): + stdout_str = ( + completed_process.stdout.decode("utf8", errors="ignore") + .strip() + .split( + "\n", + ) ) - ) - stderr_str = ( - completed_process.stderr.decode("utf8", errors="ignore") - .strip() - .split( - "\n", - ) - ) - - if len(stdout_str): self.log("info", "Script stdout:") for line in stdout_str: self.log("info", line) - if len(stderr_str): + if len(completed_process.stderr): + stderr_str = ( + completed_process.stderr.decode("utf8", errors="ignore") + .strip() + .split( + "\n", + ) + ) + self.log("warning", "Script stderr:") for line in stderr_str: self.log("warning", line)