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