From ae06b627993b912f5def32ac0a428541ef17531f Mon Sep 17 00:00:00 2001 From: disinvite Date: Sun, 19 Nov 2023 12:59:04 -0500 Subject: [PATCH] Code style, remove unneeded module and TODO --- tools/checkorder/checkorder.py | 4 ---- tools/isledecomp/isledecomp/parser/util.py | 4 ++-- 2 files changed, 2 insertions(+), 6 deletions(-) diff --git a/tools/checkorder/checkorder.py b/tools/checkorder/checkorder.py index 4f748dcd..e1235fad 100644 --- a/tools/checkorder/checkorder.py +++ b/tools/checkorder/checkorder.py @@ -1,7 +1,6 @@ import os import sys import argparse -from typing import TextIO from isledecomp.dir import ( walk_source_dir, file_is_cpp @@ -24,9 +23,6 @@ def check_file(filename: str, verbose: bool = False) -> bool: with open(filename, 'r') as f: code_blocks = find_code_blocks(f) - # TODO: Should combine these checks if/when we refactor. - # This is just for simplicity / proof of concept. - f.seek(os.SEEK_SET, 0) bad_comments = [(block.start_line, block.offset_comment) for block in code_blocks diff --git a/tools/isledecomp/isledecomp/parser/util.py b/tools/isledecomp/isledecomp/parser/util.py index 6536e2f0..c7ad91a3 100644 --- a/tools/isledecomp/isledecomp/parser/util.py +++ b/tools/isledecomp/isledecomp/parser/util.py @@ -41,8 +41,8 @@ def template_function_name(line: str) -> str: # If we don't match, you get whatever is on the line as the signature if template_match is not None: return template_match.group(1) - else: - return line + + return line def remove_trailing_comment(line: str) -> str: