[Differential] [Request, 426 lines] D4182: Remove decorator declarations
noreply at phabricator.kde.org
Wed Jan 18 09:41:03 UTC 2017
flherne created this revision.
flherne added a reviewer: brauch.
flherne set the repository for this revision to R53 KDevelop: Python Support.
Restricted Application added a subscriber: kdevelop-devel.
These seem to have been used for the builtin documentation before the current docstring solution. Currently we do nothing with decorators except a few hardcoded ones, and this declaration system is needlessly complicated.
Also fixes a bug - decorator declarations were never *removed* from a function, so removing a decorator would have no effect until renaming or moving the function.
Regression - codecompletion now adds brackets to property accesses.
Could add a temporary 'isProperty' status to functions, but I hope to implement real property support soonish.
Tested that the effects of decorators now go away when they're edited out.
Removes the decorator duchaintests, because they don't make sense without decorator declarations. The visible behaviour we actually implement (static/classmethods) is already included in several other tests.
Adds XFAIL to one completion test.
No other regressions.
R53 KDevelop: Python Support
To: flherne, brauch
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the KDevelop-devel