Removed unused parameter
authorAndrey Vlasovskikh <andrey.vlasovskikh@jetbrains.com>
Fri, 6 Nov 2015 19:46:32 +0000 (22:46 +0300)
committerAndrey Vlasovskikh <andrey.vlasovskikh@jetbrains.com>
Fri, 6 Nov 2015 19:46:32 +0000 (22:46 +0300)
21 files changed:
python/psi-api/src/com/jetbrains/python/psi/PyClass.java
python/src/com/jetbrains/python/codeInsight/PyMethodNavigationOffsetProvider.java
python/src/com/jetbrains/python/codeInsight/completion/PySuperMethodCompletionContributor.java
python/src/com/jetbrains/python/codeInsight/userSkeletons/PyUserSkeletonsClassMembersProvider.java
python/src/com/jetbrains/python/inspections/PyAbstractClassInspection.java
python/src/com/jetbrains/python/inspections/PyOldStyleClassesInspection.java
python/src/com/jetbrains/python/inspections/quickfix/AddFieldQuickFix.java
python/src/com/jetbrains/python/inspections/unresolvedReference/PyUnresolvedReferencesInspection.java
python/src/com/jetbrains/python/projectView/PyElementNode.java
python/src/com/jetbrains/python/psi/impl/PyClassImpl.java
python/src/com/jetbrains/python/psi/stubs/PyClassAttributesIndex.java
python/src/com/jetbrains/python/psi/types/PyTypeChecker.java
python/src/com/jetbrains/python/refactoring/classes/membersManager/MethodsManager.java
python/src/com/jetbrains/python/refactoring/extractmethod/PyExtractMethodUtil.java
python/src/com/jetbrains/python/testing/attest/PythonAtTestConfigurationProducer.java
python/src/com/jetbrains/python/testing/doctest/PythonDocTestUtil.java
python/src/com/jetbrains/python/testing/pytest/PyTestUtil.java
python/testSrc/com/jetbrains/python/PyControlFlowBuilderTest.java
python/testSrc/com/jetbrains/python/PyOverrideTest.java
python/testSrc/com/jetbrains/python/PyStubsTest.java
python/testSrc/com/jetbrains/python/refactoring/classes/PyDependenciesComparatorTest.java

index 36f19071610eab082b9a3b34cc6621814d8f9ad5..98c7be60a0d2c24cd37d8043d2f50b44ebf55b58 100644 (file)
@@ -105,7 +105,7 @@ public interface PyClass extends PsiNameIdentifierOwner, PyStatement, NameDefine
    * @return class methods
    */
   @NotNull
-  PyFunction[] getMethods(boolean inherited);
+  PyFunction[] getMethods();
 
   /**
    * Get class properties.
index 36c4faceaeff3b8737613aa39bd15256b7209e0e..438b79580f694bcfa78b76387f68c47ba3e71e12 100644 (file)
@@ -54,7 +54,7 @@ public class PyMethodNavigationOffsetProvider implements MethodNavigationOffsetP
       array.add(psiElement);
     }
     else if (psiElement instanceof PyClass) {
-      Collections.addAll(array, ((PyClass)psiElement).getMethods(false));
+      Collections.addAll(array, ((PyClass)psiElement).getMethods());
     }
   }
 }
index 24af1dfacd99e28f9d7db4ca0ca171c23c1ff034..f182fc83da581aae3721a732786a7d66509fce93 100644 (file)
@@ -57,13 +57,13 @@ public class PySuperMethodCompletionContributor extends CompletionContributor {
                  return;
                }
                Set<String> seenNames = new HashSet<String>();
-               for (PyFunction function : containingClass.getMethods(false)) {
+               for (PyFunction function : containingClass.getMethods()) {
                  seenNames.add(function.getName());
                }
                LanguageLevel languageLevel = LanguageLevel.forElement(parameters.getOriginalFile());
                seenNames.addAll(PyNames.getBuiltinMethods(languageLevel).keySet());
                for (PyClass ancestor : containingClass.getAncestorClasses(null)) {
-                 for (PyFunction superMethod : ancestor.getMethods(false)) {
+                 for (PyFunction superMethod : ancestor.getMethods()) {
                    if (!seenNames.contains(superMethod.getName())) {
                      String text = superMethod.getName() + superMethod.getParameterList().getText();
                      LookupElementBuilder element = LookupElementBuilder.create(text);
index c7512d3f11b59177d85144fbb23c227f510c7107..387aa2f81307fc23fc602b7f89515c3811dce52d 100644 (file)
@@ -83,7 +83,7 @@ public class PyUserSkeletonsClassMembersProvider extends PyClassMembersProviderB
 
   public static Collection<PyCustomMember> getClassMembers(@NotNull PyClass cls, boolean isDefinition) {
     final List<PyCustomMember> result = new ArrayList<PyCustomMember>();
-    for (PyFunction function : cls.getMethods(false)) {
+    for (PyFunction function : cls.getMethods()) {
       final String name = function.getName();
       final PyUtil.MethodFlags methodFlags = PyUtil.MethodFlags.of(function);
       final boolean instanceMethod = methodFlags == null || methodFlags.isInstanceMethod();
index 481db69e0594eb3ae5ea6bffb010e71f91f1a731..8873fe5228c08e681c1fcc6df9cb6d8381ae1a7b 100644 (file)
@@ -91,7 +91,7 @@ public class PyAbstractClassInspection extends PyInspection {
           return true;
         }
       }
-      for (PyFunction method : pyClass.getMethods(false)) {
+      for (PyFunction method : pyClass.getMethods()) {
         if (PyUtil.isDecoratedAsAbstract(method)) {
           return true;
         }
index 532c5a5b4112006920d853ce443c19dd2fb00086..a866b36281e2989b5797e2092162722c859de1bf 100644 (file)
@@ -74,7 +74,7 @@ public class PyOldStyleClassesInspection extends PyInspection {
             registerProblem(attr, "Old-style class contains __slots__ definition", ProblemHighlightType.GENERIC_ERROR_OR_WARNING, null, quickFixes.toArray(new LocalQuickFix[quickFixes.size()]));
           }
         }
-        for (PyFunction attr : node.getMethods(false)) {
+        for (PyFunction attr : node.getMethods()) {
           if ("__getattribute__".equals(attr.getName())) {
             final ASTNode nameNode = attr.getNameNode();
             assert nameNode != null;
index 8029597a540594d113267ebe1a87a52c336ee46b..5156205aa82ffa9338b1b7f4f8b830d4aee7dbfd 100644 (file)
@@ -163,7 +163,7 @@ public class AddFieldQuickFix implements LocalQuickFix {
         appendToMethod(newInit, callback);
 
         PsiElement addAnchor = null;
-        PyFunction[] meths = cls.getMethods(false);
+        PyFunction[] meths = cls.getMethods();
         if (meths.length > 0) addAnchor = meths[0].getPrevSibling();
         PyStatementList clsContent = cls.getStatementList();
         newInit = (PyFunction) clsContent.addAfter(newInit, addAnchor);
index d0b59e3f7fac3011454be04757181c8c46287b7b..1bfe00a95fa7106990931c70d7f51495726f4bc3 100644 (file)
@@ -888,7 +888,7 @@ public class PyUnresolvedReferencesInspection extends PyInspection {
             }
           }
         }
-        for (PyFunction method : containedClass.getMethods(false)) {
+        for (PyFunction method : containedClass.getMethods()) {
           if (expr.getText().equals(method.getName())) {
             actions.add(new UnresolvedReferenceAddSelfQuickFix(expr, qualifier));
           }
index dcf0088275dcfcfad68707b89bad9f2fba76715c..17e28cf102a1c2c186f3f8f8bdd3515686be4ed2 100644 (file)
@@ -50,7 +50,7 @@ public class PyElementNode extends BasePsiNode<PyElement> {
       for (PyClass aClass : pyClass.getNestedClasses()) {
         result.add(new PyElementNode(myProject, aClass, getSettings()));
       }
-      for (PyFunction function : pyClass.getMethods(false)) {
+      for (PyFunction function : pyClass.getMethods()) {
         result.add(new PyElementNode(myProject, function, getSettings()));
       }
       return result;
index e421b5392ecd9bcaa74b5b5cdee9ca522d977d9c..3fd8de69e7b5d49020ee8e64db6c7685373bf283 100644 (file)
@@ -463,7 +463,7 @@ public class PyClassImpl extends PyBaseElementImpl<PyClassStub> implements PyCla
 
   @Override
   @NotNull
-  public PyFunction[] getMethods(final boolean inherited) {
+  public PyFunction[] getMethods() {
     return getClassChildren(PythonDialectsTokenSetProvider.INSTANCE.getFunctionDeclarationTokens(), PyFunction.ARRAY_FACTORY);
   }
 
@@ -587,7 +587,7 @@ public class PyClassImpl extends PyBaseElementImpl<PyClassStub> implements PyCla
     // look at @property decorators
     Map<String, List<PyFunction>> grouped = new HashMap<String, List<PyFunction>>();
     // group suitable same-named methods, each group defines a property
-    for (PyFunction method : getMethods(false)) {
+    for (PyFunction method : getMethods()) {
       final String methodName = method.getName();
       if (name == null || name.equals(methodName)) {
         List<PyFunction> bucket = grouped.get(methodName);
@@ -899,7 +899,7 @@ public class PyClassImpl extends PyBaseElementImpl<PyClassStub> implements PyCla
   private boolean visitMethods(Processor<PyFunction> processor,
                               boolean inherited,
                               boolean skipClassObj, TypeEvalContext context) {
-    PyFunction[] methods = getMethods(false);
+    PyFunction[] methods = getMethods();
     if (!ContainerUtil.process(methods, processor)) return false;
     if (inherited) {
       for (PyClass ancestor : getAncestorClasses(context)) {
@@ -1013,7 +1013,7 @@ public class PyClassImpl extends PyBaseElementImpl<PyClassStub> implements PyCla
       collectInstanceAttributes(initMethod, result);
     }
     Set<String> namesInInit = new HashSet<String>(result.keySet());
-    final PyFunction[] methods = getMethods(false);
+    final PyFunction[] methods = getMethods();
     for (PyFunction method : methods) {
       if (!PyNames.INIT.equals(method.getName())) {
         collectInstanceAttributes(method, result, namesInInit);
index d371ecc7b4427f34077d1c5ab8124176672ab073..62036e848a6c040e937e00794309d76d8b750b38 100644 (file)
@@ -41,7 +41,7 @@ public class PyClassAttributesIndex extends StringStubIndexExtension<PyClass> {
   public static List<String> getAllDeclaredAttributeNames(@NotNull PyClass pyClass) {
     final List<PsiNamedElement> members = ContainerUtil.<PsiNamedElement>concat(pyClass.getInstanceAttributes(),
                                                                                 pyClass.getClassAttributes(),
-                                                                                Arrays.asList(pyClass.getMethods(false)));
+                                                                                Arrays.asList(pyClass.getMethods()));
 
     return ContainerUtil.mapNotNull(members, new Function<PsiNamedElement, String>() {
       @Override
index 09aab39c07596d13b9a667f85bd636c81e11211e..ae0450ecee0acd4e9b8f0a4769ca6ab459972007 100644 (file)
@@ -230,7 +230,7 @@ public class PyTypeChecker {
                                                 boolean isDefinition,
                                                 @NotNull TypeEvalContext context) {
     final Set<String> attributes = new HashSet<String>();
-    for (PyFunction function : cls.getMethods(false)) {
+    for (PyFunction function : cls.getMethods()) {
       attributes.add(function.getName());
     }
     for (PyTargetExpression instanceAttribute : cls.getInstanceAttributes()) {
index 1ad4f03c53cfb732bc1e330303af19e362a39761..4c5996cc6ddd0ff3509d85221b88de795b272a4c 100644 (file)
@@ -59,7 +59,7 @@ class MethodsManager extends MembersManager<PyFunction> {
 
   @Override
   public boolean hasConflict(@NotNull final PyFunction member, @NotNull final PyClass aClass) {
-    return NamePredicate.hasElementWithSameName(member, Arrays.asList(aClass.getMethods(false)));
+    return NamePredicate.hasElementWithSameName(member, Arrays.asList(aClass.getMethods()));
   }
 
   @NotNull
@@ -79,7 +79,7 @@ class MethodsManager extends MembersManager<PyFunction> {
   @NotNull
   @Override
   protected List<? extends PyElement> getMembersCouldBeMoved(@NotNull final PyClass pyClass) {
-    return FluentIterable.from(Arrays.asList(pyClass.getMethods(false))).filter(new NamelessFilter<PyFunction>()).filter(NO_PROPERTIES).toList();
+    return FluentIterable.from(Arrays.asList(pyClass.getMethods())).filter(new NamelessFilter<PyFunction>()).filter(NO_PROPERTIES).toList();
   }
 
   @Override
index 9a7161c8d057bf3dcc8c6737e0ed03a1961dc1d1..21c800a08ee49c7f85e9080a20bdc875ab625d00 100644 (file)
@@ -201,7 +201,7 @@ public class PyExtractMethodUtil {
       scope.add(owner);
       final PyClass containingClass = ((PyFunction)owner).getContainingClass();
       if (containingClass != null) {
-        for (PyFunction function : containingClass.getMethods(false)) {
+        for (PyFunction function : containingClass.getMethods()) {
           if (!function.equals(owner) && !function.equals(generatedMethod)) {
             scope.add(function);
           }
index 40115fcdb343e521eff4127b500d99c3b9dcf08c..ecf72053f29f3ed775923d41bb2b66b946357218 100644 (file)
@@ -60,7 +60,7 @@ public class PythonAtTestConfigurationProducer extends
   }
 
   private static boolean hasTestFunction(@NotNull final PyClass pyClass) {
-    PyFunction[] methods = pyClass.getMethods(false);
+    PyFunction[] methods = pyClass.getMethods();
     for (PyFunction function : methods) {
       PyDecoratorList decorators = function.getDecoratorList();
       if (decorators == null) continue;
index 351bbc01fd343fc2d16fae71a2a2dc7679037a1b..e0360ae5a6f564a066b38911ccc3de5528603c2a 100644 (file)
@@ -58,7 +58,7 @@ public class PythonDocTestUtil {
   }
 
   public static boolean isDocTestClass(PyClass pyClass) {
-    for (PyFunction cls : pyClass.getMethods(false)) {
+    for (PyFunction cls : pyClass.getMethods()) {
       if (isDocTestFunction(cls)) {
         return true;
       }
index 4bc36647257c74a7fc30592a036febaf1b2a56e5..a888e1292da9add4b46457716c74489eb37f63df 100644 (file)
@@ -83,7 +83,7 @@ public class PyTestUtil {
     if (className == null) return false;
     final String name = className.toLowerCase();
     if (name.startsWith("test")) {
-      for (PyFunction cls : pyClass.getMethods(false)) {
+      for (PyFunction cls : pyClass.getMethods()) {
         if (isPyTestFunction(cls)) {
           return true;
         }
index c5e489c0648525a42d98f04012cfac00d05e7b9a..203b462c5a44719040bc5f39761e89fcd4638c76 100644 (file)
@@ -205,7 +205,7 @@ public class PyControlFlowBuilderTest extends LightMarkedTestCase {
     configureByFile(testName + ".py");
     final String fullPath = getTestDataPath() + testName + ".txt";
     final PyClass pyClass = ((PyFile) myFile).getTopLevelClasses().get(0);
-    final ControlFlow flow = ControlFlowCache.getControlFlow(pyClass.getMethods(false)[0]);
+    final ControlFlow flow = ControlFlowCache.getControlFlow(pyClass.getMethods()[0]);
     check(fullPath, flow);
   }
 
@@ -214,7 +214,7 @@ public class PyControlFlowBuilderTest extends LightMarkedTestCase {
     configureByFile(testName + ".py");
     final String fullPath = getTestDataPath() + testName + ".txt";
     final PyClass pyClass = ((PyFile) myFile).getTopLevelClasses().get(0);
-    final ControlFlow flow = ControlFlowCache.getControlFlow(pyClass.getMethods(false)[0]);
+    final ControlFlow flow = ControlFlowCache.getControlFlow(pyClass.getMethods()[0]);
     check(fullPath, flow);
   }
 
index e89eb6c6bb7c89ae8bb97509f156396503460e7a..53a5d7f55d12d6e7ac57859c4f931ba2ae7a8f12 100644 (file)
@@ -34,7 +34,7 @@ import java.util.Collections;
 public class PyOverrideTest extends PyTestCase {
   private void doTest() {
     myFixture.configureByFile("override/" + getTestName(true) + ".py");
-    PyFunction toOverride = getTopLevelClass(0).getMethods(false) [0];
+    PyFunction toOverride = getTopLevelClass(0).getMethods() [0];
     PyOverrideImplementUtil.overrideMethods(myFixture.getEditor(), getTopLevelClass(1),
                                             Collections.singletonList(new PyMethodMember(toOverride)), false);
     myFixture.checkResultByFile("override/" + getTestName(true) + "_after.py", true);
@@ -91,7 +91,7 @@ public class PyOverrideTest extends PyTestCase {
 
   public void testInnerClass() {  // PY-10976
     myFixture.configureByFile("override/" + getTestName(true) + ".py");
-    PyFunction toOverride = getTopLevelClass(0).getMethods(false)[0];
+    PyFunction toOverride = getTopLevelClass(0).getMethods()[0];
     PyClass pyClass = getTopLevelClass(1).getNestedClasses()[0];
     PyOverrideImplementUtil.overrideMethods(myFixture.getEditor(), pyClass,
                                             Collections.singletonList(new PyMethodMember(toOverride)), false);
@@ -100,7 +100,7 @@ public class PyOverrideTest extends PyTestCase {
 
   public void testInnerFunctionClass() {
     myFixture.configureByFile("override/" + getTestName(true) + ".py");
-    PyFunction toOverride = getTopLevelClass(0).getMethods(false)[0];
+    PyFunction toOverride = getTopLevelClass(0).getMethods()[0];
     final PsiElement element = myFixture.getElementAtCaret();
     PyOverrideImplementUtil.overrideMethods(myFixture.getEditor(), PsiTreeUtil.getParentOfType(element, PyClass.class, false),
                                             Collections.singletonList(new PyMethodMember(toOverride)), false);
@@ -120,7 +120,7 @@ public class PyOverrideTest extends PyTestCase {
 
   public void testImplement() {
     myFixture.configureByFile("override/" + getTestName(true) + ".py");
-    PyFunction toImplement = getTopLevelClass(0).getMethods(false)[1];
+    PyFunction toImplement = getTopLevelClass(0).getMethods()[1];
     PyOverrideImplementUtil.overrideMethods(myFixture.getEditor(), getTopLevelClass(1),
                                             Collections.singletonList(new PyMethodMember(toImplement)), true);
     myFixture.checkResultByFile("override/" + getTestName(true) + "_after.py", true);
index f553adf47d154503fb2c223c3526189c9ceeb6ac..6b63660f8be5ccb796ac69173d2e5dd43422dd63 100644 (file)
@@ -70,7 +70,7 @@ public class PyStubsTest extends PyTestCase {
       assertEquals("staticField", attrs.get(0).getName());
       assertTrue(attrs.get(0).getAssignedQName().matches("deco"));
 
-      final PyFunction[] methods = pyClass.getMethods(false);
+      final PyFunction[] methods = pyClass.getMethods();
       assertEquals(2, methods.length);
       assertEquals("__init__", methods [0].getName());
       assertEquals("fooFunction", methods [1].getName());
@@ -117,7 +117,7 @@ public class PyStubsTest extends PyTestCase {
       Property prop = pyClass.findProperty("value", true, null);
       Maybe<PyCallable> maybe_function = prop.getGetter();
       assertTrue(maybe_function.isDefined());
-      assertEquals(pyClass.getMethods(false)[0], maybe_function.value());
+      assertEquals(pyClass.getMethods()[0], maybe_function.value());
 
       Property setvalueProp = pyClass.findProperty("setvalue", true, null);
       Maybe<PyCallable> setter = setvalueProp.getSetter();
@@ -130,10 +130,10 @@ public class PyStubsTest extends PyTestCase {
       prop = pyClass.findProperty("x", true, null);
       maybe_function = prop.getGetter();
       assertTrue(maybe_function.isDefined());
-      assertEquals(pyClass.getMethods(false)[0], maybe_function.value());
+      assertEquals(pyClass.getMethods()[0], maybe_function.value());
       maybe_function = prop.getSetter();
       assertTrue(maybe_function.isDefined());
-      assertEquals(pyClass.getMethods(false)[1], maybe_function.value());
+      assertEquals(pyClass.getMethods()[1], maybe_function.value());
 
       // ...and the juice:
       assertNotParsed(file);
@@ -332,7 +332,7 @@ public class PyStubsTest extends PyTestCase {
   public void testWrappedStaticMethod() {
     final PyFileImpl file = (PyFileImpl) getTestFile();
     final PyClass pyClass = file.getTopLevelClasses().get(0);
-    final PyFunction[] methods = pyClass.getMethods(false);
+    final PyFunction[] methods = pyClass.getMethods();
     assertEquals(1, methods.length);
     final PyFunction.Modifier modifier = methods[0].getModifier();
     assertEquals(PyFunction.Modifier.STATICMETHOD, modifier);
index b3be3825f970a8f8378203c15d68c1a324015691..ba952c3f24564e12a4be7382181e0b9b030f6860 100644 (file)
@@ -22,7 +22,7 @@ public class PyDependenciesComparatorTest extends PyTestCase {
 
     @SuppressWarnings("ConstantConditions") // Can't be null (class has docstring)
     PsiElement docStringExpression = clazz.getDocStringExpression().getParent();
-    PyFunction method = clazz.getMethods(false)[0];
+    PyFunction method = clazz.getMethods()[0];
     PsiElement classField = clazz.getClassAttributes().get(0).getParent();
 
     final List<PyStatement> elementList = new ArrayList<PyStatement>();