10052
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
1
|
# -*- coding: utf-8 -*- |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
2
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
3
|
# Copyright (c) 2023 Detlev Offenbach <detlev@die-offenbachs.de> |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
4
|
# |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
5
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
6
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
7
|
Module implementing a checker for unused arguments, variables, ... . |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
8
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
9
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
10
|
import ast |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
11
|
import copy |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
12
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
13
|
import AstUtilities |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
14
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
15
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
16
|
class UnusedChecker: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
17
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
18
|
Class implementing a checker for unused arguments, variables, ... . |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
19
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
20
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
21
|
Codes = [ |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
22
|
## Unused Arguments |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
23
|
"U100", |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
24
|
"U101", |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
25
|
] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
26
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
27
|
def __init__(self, source, filename, tree, select, ignore, expected, repeat, args): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
28
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
29
|
Constructor |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
30
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
31
|
@param source source code to be checked |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
32
|
@type list of str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
33
|
@param filename name of the source file |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
34
|
@type str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
35
|
@param tree AST tree of the source code |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
36
|
@type ast.Module |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
37
|
@param select list of selected codes |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
38
|
@type list of str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
39
|
@param ignore list of codes to be ignored |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
40
|
@type list of str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
41
|
@param expected list of expected codes |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
42
|
@type list of str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
43
|
@param repeat flag indicating to report each occurrence of a code |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
44
|
@type bool |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
45
|
@param args dictionary of arguments for the various checks |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
46
|
@type dict |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
47
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
48
|
self.__select = tuple(select) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
49
|
self.__ignore = ("",) if select else tuple(ignore) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
50
|
self.__expected = expected[:] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
51
|
self.__repeat = repeat |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
52
|
self.__filename = filename |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
53
|
self.__source = source[:] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
54
|
self.__tree = copy.deepcopy(tree) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
55
|
self.__args = args |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
56
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
57
|
### parameters for unused arguments checks |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
58
|
##self.__ignoreAbstract "IgnoreAbstract": False, |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
59
|
##self.__ignoreOverload "IgnoreOverload": False, |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
60
|
##self.__ignoreOverride "IgnoreOverride": False, |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
61
|
##self.__ignoreStubs "IgnoreStubs": False, |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
62
|
##self.__ignoreVariadicNames "IgnoreVariadicNames": False, |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
63
|
##self.__ignoreLambdas "IgnoreLambdas": False, |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
64
|
##self.__ignoreNestedFunctions "IgnoreNestedFunctions": False, |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
65
|
##self.__ignoreDunderMethods "IgnoreDunderMethods": False, |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
66
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
67
|
# statistics counters |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
68
|
self.counters = {} |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
69
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
70
|
# collection of detected errors |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
71
|
self.errors = [] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
72
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
73
|
checkersWithCodes = [ |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
74
|
(self.__checkUnusedArguments, ("U100", "U101")), |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
75
|
] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
76
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
77
|
self.__checkers = [] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
78
|
for checker, codes in checkersWithCodes: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
79
|
if any(not (code and self.__ignoreCode(code)) for code in codes): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
80
|
self.__checkers.append(checker) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
81
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
82
|
def __ignoreCode(self, code): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
83
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
84
|
Private method to check if the message code should be ignored. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
85
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
86
|
@param code message code to check for |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
87
|
@type str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
88
|
@return flag indicating to ignore the given code |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
89
|
@rtype bool |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
90
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
91
|
return code.startswith(self.__ignore) and not code.startswith(self.__select) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
92
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
93
|
def __error(self, lineNumber, offset, code, *args): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
94
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
95
|
Private method to record an issue. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
96
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
97
|
@param lineNumber line number of the issue |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
98
|
@type int |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
99
|
@param offset position within line of the issue |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
100
|
@type int |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
101
|
@param code message code |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
102
|
@type str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
103
|
@param args arguments for the message |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
104
|
@type list |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
105
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
106
|
if self.__ignoreCode(code): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
107
|
return |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
108
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
109
|
if code in self.counters: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
110
|
self.counters[code] += 1 |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
111
|
else: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
112
|
self.counters[code] = 1 |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
113
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
114
|
# Don't care about expected codes |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
115
|
if code in self.__expected: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
116
|
return |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
117
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
118
|
if code and (self.counters[code] == 1 or self.__repeat): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
119
|
# record the issue with one based line number |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
120
|
self.errors.append( |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
121
|
{ |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
122
|
"file": self.__filename, |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
123
|
"line": lineNumber + 1, |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
124
|
"offset": offset, |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
125
|
"code": code, |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
126
|
"args": args, |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
127
|
} |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
128
|
) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
129
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
130
|
def run(self): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
131
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
132
|
Public method to check the given source against miscellaneous |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
133
|
conditions. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
134
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
135
|
if not self.__filename: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
136
|
# don't do anything, if essential data is missing |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
137
|
return |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
138
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
139
|
if not self.__checkers: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
140
|
# don't do anything, if no codes were selected |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
141
|
return |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
142
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
143
|
for check in self.__checkers: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
144
|
check() |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
145
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
146
|
####################################################################### |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
147
|
## Unused Arguments |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
148
|
## |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
149
|
## adapted from: flake8-unused-arguments v0.0.13 |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
150
|
####################################################################### |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
151
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
152
|
def __checkUnusedArguments(self): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
153
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
154
|
Private method to check function and method definitions for unused arguments. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
155
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
156
|
finder = FunctionFinder(self.__args["IgnoreNestedFunctions"]) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
157
|
finder.visit(self.__tree) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
158
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
159
|
for functionNode in finder.functionNodes(): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
160
|
decoratorNames = set(self.__getDecoratorNames(functionNode)) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
161
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
162
|
# ignore overload functions, it's not a surprise when they're empty |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
163
|
if self.__args["IgnoreOverload"] and "overload" in decoratorNames: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
164
|
continue |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
165
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
166
|
# ignore overridden functions |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
167
|
if self.__args["IgnoreOverride"] and "override" in decoratorNames: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
168
|
continue |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
169
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
170
|
# ignore abstractmethods, it's not a surprise when they're empty |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
171
|
if self.__args["IgnoreAbstract"] and "abstractmethod" in decoratorNames: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
172
|
continue |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
173
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
174
|
# ignore Qt slot methods |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
175
|
if self.__args["IgnoreSlotMethods"] and ( |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
176
|
"pyqtSlot" in decoratorNames or "Slot" in decoratorNames |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
177
|
): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
178
|
continue |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
179
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
180
|
# ignore stub functions |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
181
|
if self.__args["IgnoreStubs"] and self.__isStubFunction(functionNode): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
182
|
continue |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
183
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
184
|
# ignore lambdas |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
185
|
if self.__args["IgnoreLambdas"] and isinstance(functionNode, ast.Lambda): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
186
|
continue |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
187
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
188
|
# ignore __double_underscore_methods__() |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
189
|
if self.__args["IgnoreDunderMethods"] and self.__isDunderMethod( |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
190
|
functionNode |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
191
|
): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
192
|
continue |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
193
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
194
|
for i, argument in self.__getUnusedArguments(functionNode): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
195
|
name = argument.arg |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
196
|
if self.__args["IgnoreVariadicNames"]: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
197
|
if ( |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
198
|
functionNode.args.vararg |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
199
|
and functionNode.args.vararg.arg == name |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
200
|
): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
201
|
continue |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
202
|
if functionNode.args.kwarg and functionNode.args.kwarg.arg == name: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
203
|
continue |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
204
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
205
|
# ignore self or whatever the first argument is for a classmethod |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
206
|
if i == 0 and ( |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
207
|
name in ("self", "cls") or "classmethod" in decoratorNames |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
208
|
): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
209
|
continue |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
210
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
211
|
lineNumber = argument.lineno |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
212
|
offset = argument.col_offset |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
213
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
214
|
errorCode = "U101" if name.startswith("_") else "U100" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
215
|
self.__error(lineNumber - 1, offset, errorCode, name) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
216
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
217
|
def __getDecoratorNames(self, functionNode): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
218
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
219
|
Private method to yield the decorator names of the function. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
220
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
221
|
@param functionNode reference to the node defining the function or lambda |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
222
|
@type ast.AsyncFunctionDef, ast.FunctionDef or ast.Lambda |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
223
|
@yield decorator name |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
224
|
@ytype str |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
225
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
226
|
if isinstance(functionNode, ast.Lambda): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
227
|
return |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
228
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
229
|
for decorator in functionNode.decorator_list: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
230
|
if isinstance(decorator, ast.Name): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
231
|
yield decorator.id |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
232
|
elif isinstance(decorator, ast.Attribute): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
233
|
yield decorator.attr |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
234
|
elif isinstance(decorator, ast.Call): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
235
|
if isinstance(decorator.func, ast.Name): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
236
|
yield decorator.func.id |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
237
|
else: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
238
|
yield decorator.func.attr |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
239
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
240
|
def __isStubFunction(self, functionNode): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
241
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
242
|
Private method to check, if the given function node defines a stub function. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
243
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
244
|
@param functionNode reference to the node defining the function or lambda |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
245
|
@type ast.AsyncFunctionDef, ast.FunctionDef or ast.Lambda |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
246
|
@return flag indicating a stub function |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
247
|
@rtype bool |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
248
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
249
|
if isinstance(functionNode, ast.Lambda): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
250
|
return AstUtilities.isEllipsis(functionNode.body) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
251
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
252
|
statement = functionNode.body[0] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
253
|
if isinstance(statement, ast.Expr) and AstUtilities.isString(statement.value): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
254
|
if len(functionNode.body) > 1: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
255
|
# first statement is a docstring, let's skip it |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
256
|
statement = functionNode.body[1] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
257
|
else: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
258
|
# it's a function with only a docstring, that's a stub |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
259
|
return True |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
260
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
261
|
if isinstance(statement, ast.Pass): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
262
|
return True |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
263
|
if isinstance(statement, ast.Expr) and AstUtilities.isEllipsis(statement.value): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
264
|
return True |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
265
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
266
|
if isinstance(statement, ast.Raise): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
267
|
# like 'raise NotImplementedError()' |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
268
|
if ( |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
269
|
isinstance(statement.exc, ast.Call) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
270
|
and hasattr(statement.exc.func, "id") |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
271
|
and statement.exc.func.id == "NotImplementedError" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
272
|
): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
273
|
return True |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
274
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
275
|
# like 'raise NotImplementedError' |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
276
|
elif ( |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
277
|
isinstance(statement.exc, ast.Name) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
278
|
and hasattr(statement.exc, "id") |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
279
|
and statement.exc.id == "NotImplementedError" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
280
|
): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
281
|
return True |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
282
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
283
|
return False |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
284
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
285
|
def __isDunderMethod(self, functionNode): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
286
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
287
|
Private method to check, if the function node defines a special function. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
288
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
289
|
@param functionNode reference to the node defining the function or lambda |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
290
|
@type ast.AsyncFunctionDef, ast.FunctionDef or ast.Lambda |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
291
|
@return flag indicating a special function |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
292
|
@rtype bool |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
293
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
294
|
if isinstance(functionNode, ast.Lambda): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
295
|
return False |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
296
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
297
|
if not hasattr(functionNode, "name"): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
298
|
return False |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
299
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
300
|
name = functionNode.name |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
301
|
return len(name) > 4 and name.startswith("__") and name.endswith("__") |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
302
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
303
|
def __getUnusedArguments(self, functionNode): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
304
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
305
|
Private method to get a list of unused arguments of the given function. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
306
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
307
|
@param functionNode reference to the node defining the function or lambda |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
308
|
@type ast.AsyncFunctionDef, ast.FunctionDef or ast.Lambda |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
309
|
@return list of tuples of the argument position and the argument |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
310
|
@rtype list of tuples of (int, ast.arg) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
311
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
312
|
arguments = list(enumerate(self.__getArguments(functionNode))) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
313
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
314
|
class NameFinder(ast.NodeVisitor): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
315
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
316
|
Class to find the used argument names. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
317
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
318
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
319
|
def visit_Name(self, name): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
320
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
321
|
Public method to check a Name node. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
322
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
323
|
@param name reference to the name node to be checked |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
324
|
@type ast.Name |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
325
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
326
|
nonlocal arguments |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
327
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
328
|
if isinstance(name.ctx, ast.Store): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
329
|
return |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
330
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
331
|
arguments = [ |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
332
|
(argIndex, arg) for argIndex, arg in arguments if arg.arg != name.id |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
333
|
] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
334
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
335
|
NameFinder().visit(functionNode) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
336
|
return arguments |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
337
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
338
|
def __getArguments(self, functionNode): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
339
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
340
|
Private method to get all argument names of the given function. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
341
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
342
|
@param functionNode reference to the node defining the function or lambda |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
343
|
@type ast.AsyncFunctionDef, ast.FunctionDef or ast.Lambda |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
344
|
@return list of argument names |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
345
|
@rtype list of ast.arg |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
346
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
347
|
args = functionNode.args |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
348
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
349
|
orderedArguments = [] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
350
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
351
|
# plain old args |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
352
|
orderedArguments.extend(args.args) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
353
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
354
|
# *arg name |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
355
|
if args.vararg is not None: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
356
|
orderedArguments.append(args.vararg) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
357
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
358
|
# *, key, word, only, args |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
359
|
orderedArguments.extend(args.kwonlyargs) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
360
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
361
|
# **kwarg name |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
362
|
if args.kwarg is not None: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
363
|
orderedArguments.append(args.kwarg) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
364
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
365
|
return orderedArguments |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
366
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
367
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
368
|
class FunctionFinder(ast.NodeVisitor): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
369
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
370
|
Class to find all defined functions and methods. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
371
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
372
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
373
|
def __init__(self, onlyTopLevel=False): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
374
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
375
|
Constructor |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
376
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
377
|
@param onlyTopLevel flag indicating to search for top level functions only |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
378
|
(defaults to False) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
379
|
@type bool (optional) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
380
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
381
|
super().__init__() |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
382
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
383
|
self.__functions = [] |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
384
|
self.__onlyTopLevel = onlyTopLevel |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
385
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
386
|
def functionNodes(self): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
387
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
388
|
Public method to get the list of detected functions and lambdas. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
389
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
390
|
@return list of detected functions and lambdas |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
391
|
@rtype list of ast.AsyncFunctionDef, ast.FunctionDef or ast.Lambda |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
392
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
393
|
return self.__functions |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
394
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
395
|
def __visitFunctionTypes(self, functionNode): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
396
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
397
|
Private method to handle an AST node defining a function or lambda. |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
398
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
399
|
@param functionNode reference to the node defining a function or lambda |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
400
|
@type ast.AsyncFunctionDef, ast.FunctionDef or ast.Lambda |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
401
|
""" |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
402
|
self.__functions.append(functionNode) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
403
|
if not self.__onlyTopLevel: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
404
|
if isinstance(functionNode, ast.Lambda): |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
405
|
self.visit(functionNode.body) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
406
|
else: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
407
|
for obj in functionNode.body: |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
408
|
self.visit(obj) |
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
409
|
|
Detlev Offenbach <detlev@die-offenbachs.de>
parents:
diff
changeset
|
410
|
visit_AsyncFunctionDef = visit_FunctionDef = visit_Lambda = __visitFunctionTypes |