Skip to content
Snippets Groups Projects
Commit b56caec8 authored by Francis Herne's avatar Francis Herne
Browse files

Make ExpressionVisitor::visitCall() clearer.

Rename `checkForDecorators()` to `docstringTypeOverride()`,
 because it doesn't do anything involving decorators. (!)

Move the code choosing between class and function types to `visitCall()`,
 because it's nothing to do with docstring contents and made the
 different code paths very hard to follow.

Make it return the type it produces, rather than calling `encounter()`
 itself, for clarity.

Use the new `isAlias` argument of `functionForCalled` instead of
 some code to do the same thing.

Test `init_class_no_decl` is (still) XFAIL, we can't fix
 that with the current system for class types.

Test `call_class_no_decl` worked before, but I broke that behaviour in
 an earlier version of this patch and realised there wasn't a test.

Differential Revision: https://phabricator.kde.org/D3524
parent 7eb15adf
No related branches found
No related tags found
No related merge requests found
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment