D18218: Properly display argument names of template functions

[Thomas Schöps]

thomassc added a comment.


  Yes, the behavior of this patch is as described by Milian.
  
  Regarding commit rights, I applied for them now, but feel free to commit directly in case you want to have it in @aacid .
  
  Regarding showing the template arguments in the tooltip as well, KDevelop's behavior seems a bit inconsistent at the moment. If you have for example this template class:
  
    template <int A>
    class TemplateClass {
      int SomeFunction() {
        return 0;
      }
    };
  
  Hovering over "TemplateClass" will not show any template arguments. However, hovering over "SomeFunction" will show "Container: TemplateClass<A>" in the tooltip. This is because the template identifier info is explicitly removed from the main tooltip display. But even if disabling this, it will only show the names of the template arguments, not the types. So, I might look into this when I find the time for it.

REPOSITORY
  R32 KDevelop

BRANCH
  fix_template_function_arguments_3 (branched from master)

REVISION DETAIL
  https://phabricator.kde.org/D18218

To: thomassc, #kdevelop, mwolff
Cc: aacid, mwolff, kdevelop-devel, glebaccon, hase, antismap, iodelay, geetamc, Pilzschaf, akshaydeo, surgenight, arrowd
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20190120/4d1b025c/attachment.html>