Skip to content

Commit

Permalink
wmllint: cleanup two useless lines
Browse files Browse the repository at this point in the history
As far as I can tell this lines do nothing.
1. `file_handle = None` is NOT how to close a file in python
2. The file handle is already closed due to `with open() as file_handle:`
  • Loading branch information
hexagonrecursion authored and Elvish-Hunter committed Feb 2, 2021
1 parent 2296a17 commit bbf3840
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions data/tools/wmllint
Expand Up @@ -3431,7 +3431,6 @@ In your case, your system interprets your arguments as:
failed_any_dirs = True
print("wmllint: skipping non-existent path %s" % directory)
continue
ofp = None
for fn in allcfgfiles(directory):
if verbose >= 2:
print(fn + ":")
Expand Down Expand Up @@ -3505,7 +3504,6 @@ In your case, your system interprets your arguments as:
failed_any_dirs = True
print("wmllint: skipping non-existent path %s" % directory)
continue
ofp = None
for fn in allcfgfiles(directory):
if verbose >= 2:
print(fn + ":")
Expand Down

0 comments on commit bbf3840

Please sign in to comment.