[Differential] [Request, 19 lines] D3422: Pass AbstractType::Ptr, not Declaration*, to accessAttribute()
flherne (Francis Herne)
noreply at phabricator.kde.org
Fri Nov 18 20:31:28 UTC 2016
flherne created this revision.
flherne set the repository for this revision to rKDEVPYTHON KDev-python.
Restricted Application added a subscriber: kdevelop-devel.
REVISION SUMMARY
All but one caller had to look up the declaration only to have it converted back, so the previous version was pointless anyway.
This allows subscripting of expressions (inc. other subscripts), when the subscripted type isn't a ListType/IndexedContainer.
BUG: 359912
TEST PLAN
No test regressions or new compile warnings.
Fixes both failing examples from https://bugs.kde.org/show_bug.cgi?id=359912 .
TODO: Add tests. Is there a tidier way to add larger chunks of code to tests? I'll ask on IRC.
REPOSITORY
rKDEVPYTHON KDev-python
REVISION DETAIL
https://phabricator.kde.org/D3422
AFFECTED FILES
duchain/declarationbuilder.cpp
duchain/expressionvisitor.cpp
duchain/helpers.cpp
duchain/helpers.h
EMAIL PREFERENCES
https://phabricator.kde.org/settings/panel/emailpreferences/
To: flherne
Cc: kdevelop-devel
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20161118/6b839f6f/attachment.html>
More information about the KDevelop-devel
mailing list