Review Request 124139: Added clang settings pages

Sergey Kalinichev kalinichev.so.0 at gmail.com
Sun Jul 12 10:36:42 UTC 2015


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

(Updated July 12, 2015, 1:36 p.m.)


Review request for KDevelop.


Changes
-------

Moved a lot of code to kdevelop repository, fixed mentioned issues.


Repository: kdev-clang


Description
-------

This adds two pages: One for session settings (code-completion, assistants), another one for project settings (parser command-line arguments).

Known issue: now we can set language standard in two places: DaIM - for deducing standard macros/includes and here - for the internal parser.
To solve this problem we can copy/move compilers infrastructure to kdev-clang. Also we can expose the compiler provider interface through DaIM (which is a much better solution imo). Suggestions?


Diffs (updated)
-----

  CMakeLists.txt 218cdcd 
  clangparsejob.cpp 127e0a9 
  clangsettings/CMakeLists.txt PRE-CREATION 
  clangsettings/clangsettingsmanager.h PRE-CREATION 
  clangsettings/clangsettingsmanager.cpp PRE-CREATION 
  clangsettings/sessionsettings/sessionconfig.kcfg PRE-CREATION 
  clangsettings/sessionsettings/sessionconfig.kcfgc PRE-CREATION 
  clangsettings/sessionsettings/sessionconfigskeleton.h PRE-CREATION 
  clangsettings/sessionsettings/sessionsettings.h PRE-CREATION 
  clangsettings/sessionsettings/sessionsettings.cpp PRE-CREATION 
  clangsettings/sessionsettings/sessionsettings.ui PRE-CREATION 
  clangsupport.h 8ed1ec9 
  clangsupport.cpp e22c554 
  codecompletion/context.cpp f26921f 
  duchain/CMakeLists.txt e07ef70 
  duchain/clangparsingenvironment.h 2e4ea8b 
  duchain/clangparsingenvironment.cpp 20a2dbf 
  duchain/parsesession.h 74999de 
  duchain/parsesession.cpp cfafa33 
  duchain/unknowndeclarationproblem.cpp 908a518 

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


Testing
-------


File Attachments
----------------

project settings.png
  https://git.reviewboard.kde.org/media/uploaded/files/2015/06/21/f4bb2220-dc63-478d-b423-f31434496afb__project_settings.png
session settings.png
  https://git.reviewboard.kde.org/media/uploaded/files/2015/06/21/e1f59a46-f0b4-46fa-9f19-ffca4b887cf5__session_settings.png


Thanks,

Sergey Kalinichev

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


More information about the KDevelop-devel mailing list