Review Request 117835: Now IDM (Includes/Defines manager) provides all types of includes/defines, not just user-specified. (kdevplatform)

Sergey Kalinichev kalinichev.so.0 at gmail.com
Thu May 1 11:20:58 UTC 2014


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

(Updated May 1, 2014, 3:20 p.m.)


Review request for KDevelop.


Changes
-------

Added more comments.
Also brought back i/d into IBSM. Turns out there are many plugins that depend on it: QMake, kdev-kernel and probably others. And I don't want to make them unusable...


Repository: kdevplatform


Description
-------

The changes include extending IDM interface and removing i/d notion from IBuildSystemManager interface.


Diffs (updated)
-----

  language/interfaces/idefinesandincludesmanager.h ab42444 
  project/interfaces/ibuildsystemmanager.h 241b696 
  project/interfaces/ibuildsystemmanager.cpp 74af0e7 

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


Testing
-------


Thanks,

Sergey Kalinichev

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


More information about the KDevelop-devel mailing list