diff options
| author | Claudiu Popa <cpopa@cloudbasesolutions.com> | 2015-03-14 17:50:36 +0200 |
|---|---|---|
| committer | Claudiu Popa <cpopa@cloudbasesolutions.com> | 2015-03-14 17:50:36 +0200 |
| commit | 4a5848dccdaf3a2a1f966c38a5f4a2b410b336e1 (patch) | |
| tree | 7e2dbae484de823383a40b1b193381ad8395c71d /doc | |
| parent | 0b4f87b13c34d341388f0e692f06c4b73c6ea2b0 (diff) | |
| download | pylint-4a5848dccdaf3a2a1f966c38a5f4a2b410b336e1.tar.gz | |
Added a new option for controlling the peephole optimizer in astroid.
The option ``--optimize-ast`` will control the peephole optimizer,
which is used to optimize a couple of AST subtrees. The current problem
solved by the peephole optimizer is when multiple joined strings,
with the addition operator, are encountered. If the numbers of such
strings is high enough, Pylint will then fail with a maximum recursion
depth exceeded error, due to its visitor architecture. The peephole
just transforms such calls, if it can, into the final resulting string
and this exhibit a problem, because the visit_binop method stops being
called (in the optimized AST it will be a Const node).
Diffstat (limited to 'doc')
| -rw-r--r-- | doc/faq.rst | 17 |
1 files changed, 15 insertions, 2 deletions
diff --git a/doc/faq.rst b/doc/faq.rst index a42080e..1ce9a3e 100644 --- a/doc/faq.rst +++ b/doc/faq.rst @@ -261,12 +261,25 @@ mangled sys.path. Pylint doesn't import any of the candidate modules and thus doesn't include any of import's side effects (good and bad). It traverses an AST representation of the code. -6.3 I think I found a bug in Pylint. What should I do? +6.3 Pylint keeps crashing with `Maximum recursion depth exceeded` +----------------------------------------------------------------- + +Pylint can crash with this error if you have a string in your analyzed +program, created by joining a lot of strings with the addition operator. +Due to how Pylint works, visiting nodes on a AST tree and due to how +the BinOp node is represented (the node which represents the string '1+1' +for instance), the same visit method will be called over and over again, leading +to a maximum recursion error. You can alleviate this problem by passing +the flag `--optimize-ast=y` to Pylint. This will activate an optimization +which will transform such AST subtrees into the final resulting string. +This flag is off by default. If this is not the case, please report a bug! + +6.4 I think I found a bug in Pylint. What should I do? ------------------------------------------------------- Read http://docs.pylint.org/contribute#bug-reports-feedback -6.4 I have a question about Pylint that isn't answered here. +6.5 I have a question about Pylint that isn't answered here. ------------------------------------------------------------ Read http://docs.pylint.org/contribute#mailing-lists |