Fix some issues found by lgtm.com

techtonik / python-patch

Library to parse and apply unified diffs

https://pypi.python.org/pypi/patch

106 stars 63 forks source link

Fix some issues found by lgtm.com #64

Open misery opened 4 years ago

misery commented 4 years ago

This import of module shutil is redundant, as it was previously imported on line 35.
Testing for None should use the 'is' operator.
The value assigned to local variable 'hno' is never used.

techtonik commented 3 years ago

lgtm.com is right about shutil, but I've heard from Fedora users that shutil has some permission problems. I need to ask them for testing as well.