Review Request 124745: Always attach AST to the active document

Sergey Kalinichev kalinichev.so.0 at gmail.com
Thu Aug 20 08:32:27 UTC 2015


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

(Updated Aug. 20, 2015, 11:32 a.m.)


Review request for KDevelop.


Changes
-------

*Fixed some small issues.
*Fixed DUChainUtils::standardContextForUrl to always return correct context. No need for hacks all over the place to retrieve the pinned context anymore!


Repository: kdev-clang


Description
-------

This should fix the issue when we had no code completion after restoring context from disk, if the active document wasn't modified beforehand.


Diffs (updated)
-----

  clangparsejob.h ba2d5ac 
  clangparsejob.cpp a80433e 
  clangsupport.h a047382 
  clangsupport.cpp 22b6c96 
  codecompletion/model.cpp 57902d3 
  tests/test_duchain.h 694f5a2 
  tests/test_duchain.cpp 796e3c6 

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


Testing
-------


Thanks,

Sergey Kalinichev

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdevelop-devel/attachments/20150820/4de575d3/attachment.html>


More information about the KDevelop-devel mailing list