diff --git a/analyzer/codechecker_analyzer/analysis_manager.py b/analyzer/codechecker_analyzer/analysis_manager.py index fed7965f21..2593787842 100644 --- a/analyzer/codechecker_analyzer/analysis_manager.py +++ b/analyzer/codechecker_analyzer/analysis_manager.py @@ -60,10 +60,13 @@ def print_analyzer_statistic_summary(metadata_analyzers, status, msg=None): LOG.info(" %s: %s", analyzer_type, res) -def worker_result_handler(results, metadata_tool, output_path, plist_file_name): +def worker_result_handler(results, + metadata_tool, + output_path, + plist_file_name): """ - Handle analysis results after all the analyzer threads returned. It may - merge all the plist output files into one, and print the analysis summary. + Handle analysis results after all the analyzer threads returned. It may + merge all the plist output files into one, and print the analysis summary. """ LOG.info("Merging plist files into %s", plist_file_name) if plist_file_name: @@ -72,7 +75,7 @@ def worker_result_handler(results, metadata_tool, output_path, plist_file_name): for _, _, _, _, original_plist, _ in results: original_plist = Path(original_plist) if os.path.exists(original_plist): - with open(original_plist, 'rb') as plist: + with open(original_plist, 'rb') as plist: LOG.debug(f"Merging original plist {original_plist}") plist_data.append(plistlib.load(plist)) @@ -837,7 +840,10 @@ def signal_handler(signum, _): analyzed_actions, 1, callback=lambda results: worker_result_handler( - results, metadata_tool, output_path, plist_file_name) + results, + metadata_tool, + output_path, + plist_file_name) ).get(timeout) pool.close() diff --git a/analyzer/codechecker_analyzer/buildlog/build_action.py b/analyzer/codechecker_analyzer/buildlog/build_action.py index d576fefe18..1a43fcd5a8 100644 --- a/analyzer/codechecker_analyzer/buildlog/build_action.py +++ b/analyzer/codechecker_analyzer/buildlog/build_action.py @@ -76,6 +76,6 @@ def with_attr(self, attr, value): details = {key: getattr(self, key) for key in BuildAction.__slots__} details[attr] = value return BuildAction(**details) - + def __repr__(self): return str(self) diff --git a/analyzer/codechecker_analyzer/cmd/check.py b/analyzer/codechecker_analyzer/cmd/check.py index c2381c4bbc..cd43133598 100644 --- a/analyzer/codechecker_analyzer/cmd/check.py +++ b/analyzer/codechecker_analyzer/cmd/check.py @@ -111,7 +111,7 @@ def add_arguments_to_parser(parser): "If it is not given then the results go into a " "temporary directory which will be removed after " "the analysis.") - + parser.add_argument('--plist-file-name', type=str, dest="plist_file_name", diff --git a/tools/report-converter/codechecker_report_converter/report/parser/plist.py b/tools/report-converter/codechecker_report_converter/report/parser/plist.py index 471b079ec6..e6e7811d74 100644 --- a/tools/report-converter/codechecker_report_converter/report/parser/plist.py +++ b/tools/report-converter/codechecker_report_converter/report/parser/plist.py @@ -207,7 +207,7 @@ def get_reports( return reports if not isinstance(plist, list): - plist=[plist] + plist = [plist] for sub_plist in plist: