Review Request 124969: Show popup menu with overriden methods when clicking on virtual function declaration

Oleksandr Senkovych bjsenya at gmail.com
Mon Mar 21 09:31:19 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124969/
-----------------------------------------------------------

(Updated March 21, 2016, 9:31 a.m.)


Status
------

This change has been discarded.


Review request for KDevelop and Kevin Funk.


Repository: kdevplatform


Description
-------

Items are sorted in a following order:
1. Link to function definition
2. Link to a declaration of a base class
3. Alphabetically sorted all possible overrides of a method


Diffs
-----

  plugins/contextbrowser/browsemanager.h ca6461a 
  plugins/contextbrowser/browsemanager.cpp 2574559 

Diff: https://git.reviewboard.kde.org/r/124969/diff/


Testing
-------

Currently there's two unresolved issues with it:
1. If there's too much possible overloads - menu will eat all the space on a screen.
2. maxAllowedSteps parameter is UINT_MAX, so there could be some freezes depending on project complexity. I don't have them on my machine when showing items for a really big project (clang parses it for 2-3 hours).


Thanks,

Oleksandr Senkovych

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20160321/6198bb30/attachment.html>


More information about the KDevelop-devel mailing list