https://bugs.kde.org/show_bug.cgi?id=369364

Francis Herne <m...@flherne.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
      Latest Commit|                            |https://commits.kde.org/kde
                   |                            |v-python/b36edbb6c54a462cc6
                   |                            |d72c344c0f760d5a7c2248
             Status|CONFIRMED                   |RESOLVED

--- Comment #2 from Francis Herne <m...@flherne.uk> ---
Git commit b36edbb6c54a462cc6d72c344c0f760d5a7c2248 by Francis Herne.
Committed on 14/12/2016 at 09:07.
Pushed by flherne into branch 'master'.

Skip explicit `self` argument when calling via class.

Instance methods can be called via the class:
 `Class.method(instance, arg)`

Self arguments are handled separately. With an explicit `instance`
 argument as in the example, its type was wrongly used for the next
 argument with all subsequent argtypes offset by one.

Simply skip such arguments where they exist.

Test `method_explicit_self` previously failed, now passes.

Test `parent_constructor_arg_type` previously failed.
Now it fails on the first run, but gets the correct type if reloading
 the document.
Probably related to https://bugs.kde.org/show_bug.cgi?id=306213

Other new tests passed already.
No test regressions.

M  +21   -9    duchain/declarationbuilder.cpp
M  +29   -0    duchain/tests/pyduchaintest.cpp

https://commits.kde.org/kdev-python/b36edbb6c54a462cc6d72c344c0f760d5a7c2248

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to