From 5634b3e649fed5971e85cd62a1cbacb768e8f692 Mon Sep 17 00:00:00 2001
From: jon-lee <jon-lee@sjtu.edu.cn>
Date: Sun, 2 Mar 2025 11:00:41 +0800
Subject: [PATCH] style(processors/task): fix_keyword

---
 joj3_config_generator/processers/task.py | 57 +++++++++++-------------
 1 file changed, 26 insertions(+), 31 deletions(-)

diff --git a/joj3_config_generator/processers/task.py b/joj3_config_generator/processers/task.py
index 3d6637f..6fb8074 100644
--- a/joj3_config_generator/processers/task.py
+++ b/joj3_config_generator/processers/task.py
@@ -125,39 +125,34 @@ def fix_keyword(
     task_stage: task.Stage, conf_stage: result.StageDetail
 ) -> result.StageDetail:
     keyword_parser = ["clangtidy", "keyword", "cppcheck", "cpplint"]
-    if task_stage.parsers is not None:
-        for parser in task_stage.parsers:
-            if parser in keyword_parser:
-                keyword_parser_ = next(
-                    p for p in conf_stage.parsers if p.name == parser
-                )
-                keyword_weight = []
-                if getattr(task_stage, parser, None) is not None:
-                    unique_weight = list(set(getattr(task_stage, parser).weight))
-                    for score in unique_weight:
-                        keyword_weight.append({"keywords": [], "score": score})
+    for parser in task_stage.parsers or []:
+        if parser in keyword_parser:
+            keyword_parser_ = next(p for p in conf_stage.parsers if p.name == parser)
+            keyword_weight = []
+            if getattr(task_stage, parser, None) is not None:
+                unique_weight = list(set(getattr(task_stage, parser).weight))
+                for score in unique_weight:
+                    keyword_weight.append({"keywords": [], "score": score})
 
-                    for idx, score in enumerate(unique_weight):
-                        for idx_, score_ in enumerate(
-                            getattr(task_stage, parser).weight
-                        ):
-                            if score == score_:
-                                keyword_weight[idx]["keywords"].append(
-                                    getattr(task_stage, parser).keyword[idx_]
-                                )
-                            else:
-                                continue
+                for idx, score in enumerate(unique_weight):
+                    for idx_, score_ in enumerate(getattr(task_stage, parser).weight):
+                        if score == score_:
+                            keyword_weight[idx]["keywords"].append(
+                                getattr(task_stage, parser).keyword[idx_]
+                            )
+                        else:
+                            continue
 
-                keyword_parser_.with_.update(
-                    {
-                        "matches": keyword_weight,
-                        "fullscore": 0,
-                        "minscore": -1000,
-                        "files": ["stdout", "stderr"],
-                    }
-                )
-            else:
-                continue
+            keyword_parser_.with_.update(
+                {
+                    "matches": keyword_weight,
+                    "fullscore": 0,
+                    "minscore": -1000,
+                    "files": ["stdout", "stderr"],
+                }
+            )
+        else:
+            continue
     return conf_stage