Review Request: introduce the class Configuration Compile Configuration for kconfig_compiler

Jaime Torres jtamate at gmail.com
Thu Sep 30 18:35:38 BST 2010


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/5479/
-----------------------------------------------------------

(Updated 2010-09-30 17:35:38.643873)


Review request for kdelibs.


Changes
-------

Implemented Aaron suggestions.


Summary
-------

Just create the class CfgConfig, grouping all the configuration items in one place.
Also, use an instance where any of the configuration values is needed.

Next patch: make independent the input parser and the output to c++ language.


Diffs (updated)
-----

  /trunk/KDE/kdelibs/kdecore/kconfig_compiler/kconfig_compiler.cpp 1180729 

Diff: http://svn.reviewboard.kde.org/r/5479/diff


Testing
-------

tested time ago in cygwin.
tested few minutes ago recompiling akregator.


Thanks,

Jaime

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20100930/7077a418/attachment.htm>


More information about the kde-core-devel mailing list