Merge pull request #107 from codellm-devkit/issue-105-remove-unused-parameter

rangeetpan · web-flow · commit bf44358dfeb1 · 2025-02-20T08:11:47.000-05:00
removed unused parameter issue #104
diff --git a/cldk/analysis/java/codeanalyzer/codeanalyzer.py b/cldk/analysis/java/codeanalyzer/codeanalyzer.py
@@ -313,7 +313,7 @@ def _generate_call_graph(self, using_symbol_table) -> DiGraph:
                         "type": jge.type,
                         "weight": jge.weight,
                         "calling_lines": (
-                            tsu.get_calling_lines(jge.source.method.code, jge.target.method.signature, jge.target.method.is_constructor)
+                            tsu.get_calling_lines(jge.source.method.code, jge.target.method.signature)
                             if not jge.source.method.is_implicit or not jge.target.method.is_implicit
                             else []
                         ),
diff --git a/tests/analysis/java/test_java_sitter.py b/tests/analysis/java/test_java_sitter.py
@@ -287,14 +287,14 @@ def test_get_calling_lines():
     }
 """
     # test where call is found
-    calling_lines = java_sitter.get_calling_lines(source_method_code, "hasNext", False)
+    calling_lines = java_sitter.get_calling_lines(source_method_code, "hasNext")
     assert calling_lines is not None
     assert isinstance(calling_lines, List)
     assert len(calling_lines) == 1
     assert calling_lines[0] == 7
 
     # test where call is not found
-    calling_lines = java_sitter.get_calling_lines(source_method_code, "foo", False)
+    calling_lines = java_sitter.get_calling_lines(source_method_code, "foo")
     assert calling_lines is not None
     assert isinstance(calling_lines, List)
     assert len(calling_lines) == 0
