diff --git a/platformio/builder/tools/piolib.py b/platformio/builder/tools/piolib.py index aa4b2fe6..755900da 100644 --- a/platformio/builder/tools/piolib.py +++ b/platformio/builder/tools/piolib.py @@ -128,9 +128,9 @@ class LibBuilderBase(object): self._is_dependent = False self._is_built = False - self._depbuilders = list() - self._circular_deps = list() - self._processed_files = list() + self._depbuilders = [] + self._circular_deps = [] + self._processed_files = [] # reset source filter, could be overridden with extra script self.env["SRC_FILTER"] = "" diff --git a/platformio/builder/tools/piosize.py b/platformio/builder/tools/piosize.py index 6ef8d945..7eece12a 100644 --- a/platformio/builder/tools/piosize.py +++ b/platformio/builder/tools/piosize.py @@ -220,7 +220,7 @@ def DumpSizeData(_, target, source, env): # pylint: disable=unused-argument "sections": sections, } - files = dict() + files = {} for symbol in _collect_symbols_info(env, elffile, elf_path, sections): file_path = symbol.get("file") or "unknown" if not files.get(file_path, {}): @@ -235,7 +235,7 @@ def DumpSizeData(_, target, source, env): # pylint: disable=unused-argument files[file_path]["symbols"].append(symbol) - data["memory"]["files"] = list() + data["memory"]["files"] = [] for k, v in files.items(): file_data = {"path": k} file_data.update(v) diff --git a/platformio/commands/check/command.py b/platformio/commands/check/command.py index 2d4bb109..87673f99 100644 --- a/platformio/commands/check/command.py +++ b/platformio/commands/check/command.py @@ -215,7 +215,7 @@ def print_processing_footer(result): def collect_component_stats(result): - components = dict() + components = {} def _append_defect(component, defect): if not components.get(component): @@ -250,7 +250,7 @@ def print_defects_stats(results): severity_labels = list(DefectItem.SEVERITY_LABELS.values()) severity_labels.reverse() - tabular_data = list() + tabular_data = [] for k, v in component_stats.items(): tool_defect = [v.get(s, 0) for s in severity_labels] tabular_data.append([k] + tool_defect) diff --git a/platformio/commands/check/tools/cppcheck.py b/platformio/commands/check/tools/cppcheck.py index c1fe3e3b..8dd6041a 100644 --- a/platformio/commands/check/tools/cppcheck.py +++ b/platformio/commands/check/tools/cppcheck.py @@ -64,7 +64,7 @@ class CppcheckCheckTool(CheckToolBase): if any(f not in self._buffer for f in self.defect_fields): return None - args = dict() + args = {} for field in self._buffer.split(self._field_delimiter): field = field.strip().replace('"', "") name, value = field.split("=", 1) diff --git a/platformio/package/lockfile.py b/platformio/package/lockfile.py index 8d6c837c..b04cd428 100644 --- a/platformio/package/lockfile.py +++ b/platformio/package/lockfile.py @@ -62,9 +62,9 @@ class LockFile(object): else: raise LockFileExists - self._fp = open( + self._fp = open( # pylint: disable=consider-using-with self._lock_path, mode="w", encoding="utf8" - ) # pylint: disable=consider-using-with + ) try: if LOCKFILE_CURRENT_INTERFACE == LOCKFILE_INTERFACE_FCNTL: fcntl.flock(self._fp.fileno(), fcntl.LOCK_EX | fcntl.LOCK_NB)