The new feature is fine, it's small enough. Howver, I'd appreciate if the gunzip calling could be cleaned up, such as
gunzip = subprocess.Popen(['gunzip', localPath], stdout=subprocess.PIPE, stderr=subprocess.PIPE) (output, output_err) = gunzip.communicate() status = gunzip.returncode
The new feature is fine, it's small enough. Howver, I'd appreciate if the gunzip calling could be cleaned up, such as
gunzip = subprocess. Popen([ 'gunzip' , localPath], stdout= subprocess. PIPE, stderr= subprocess. PIPE) communicate( )
(output, output_err) = gunzip.
status = gunzip.returncode