Review Request 121196: Custom Build System human readable defines and includes path.

[Salamander Purake]

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

(Updated Nov. 30, 2014, 8:04 p.m.)


Review request for KDevelop.


Changes
-------

Fixed the issues you talked about and based it off the new changes to master.


Repository: kdevelop


Description
-------

Made the defines and include path lists human readable by removing the QDataStream and QByte types from the save load steps.


Diffs (updated)
-----

  languages/plugins/custom-definesandincludes/compilerprovider/settingsmanager.h 5cb3ca0 
  languages/plugins/custom-definesandincludes/compilerprovider/settingsmanager.cpp 229f456 

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


Testing
-------

create/load a custom build system project (${project-name}.kdev4) add defines and includes path to project via in the text file and the dialog. Tested the defines lines inside the source code files and the autocompletion also in the code files.


Thanks,

Salamander Purake

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