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

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

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

--- Comment #1 from Francis Herne <m...@flherne.uk> ---
Git commit cbcde7f0136bde15858931774fb9e44a8e217a7e by Francis Herne.
Committed on 01/01/2017 at 01:03.
Pushed by flherne into branch '5.1'.

Completion hint: Correct default-argument index for non-static methods.

>From the bug:
```
class C:
    def func1(self, arg1, arg2, arg3=None, arg4=42, arg5='default'):
        pass
c = C()
c.func1(
 # argument tooltip says void func1 (arg1, arg2, arg3, [arg4, arg5])
```

Now '(arg1, arg2, [arg3, arg4, arg5])' as expected.

Still wrong in case of calling the method directly from the class:
`C.func1(c,` <- same tooltip, but now the self argument is explicit.

This was broken before too, and would require much more intrusive
 changes to fix because we only have the function declaration currently.

Quick fix for 5.1, I'm making some larger changes for master.

M  +4    -5    codecompletion/helpers.cpp

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

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

Reply via email to