[Differential] [Updated, 337 lines] D4182: Remove decorator declarations

Francis Herne noreply at phabricator.kde.org
Sun Feb 26 11:04:51 UTC 2017


flherne updated this revision to Diff 11846.
flherne edited the summary of this revision.
flherne edited the test plan for this revision.
flherne added a comment.


  Added an 'isProperty' flag to FunctionDeclaration to avoid regressing for the moment. (in fact it should be slightly better, because it understands `@foo.getter`).
  
  Left Python::ClassDeclaration in as a stub  - without the decorator decls, it now doesn't have anything, but as you say it would make sense to store strings or something similar for use in the tooltips.

REPOSITORY
  R53 KDevelop: Python Support

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D4182?vs=10310&id=11846

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

AFFECTED FILES
  codecompletion/items/functiondeclaration.cpp
  duchain/declarationbuilder.cpp
  duchain/declarationbuilder.h
  duchain/declarations/classdeclaration.cpp
  duchain/declarations/classdeclaration.h
  duchain/declarations/decorator.h
  duchain/declarations/functiondeclaration.cpp
  duchain/declarations/functiondeclaration.h
  duchain/expressionvisitor.cpp
  duchain/helpers.h
  duchain/tests/pyduchaintest.cpp
  duchain/tests/pyduchaintest.h

EMAIL PREFERENCES
  https://phabricator.kde.org/settings/panel/emailpreferences/

To: flherne, brauch
Cc: kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20170226/091a1a85/attachment.html>


More information about the KDevelop-devel mailing list