<table><tr><td style="">tcanabrava created this revision.<br />Herald added a project: Frameworks.<br />Herald added a subscriber: kde-frameworks-devel.<br />tcanabrava requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D26766">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Simplify If-Else chain inside of defaultValue function</p>

<p>Use a type collection to verify which value we should return<br />
to the code generator. This code is slower than the original one<br />
but it's more maintenable.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R237 KConfig</div></div></div><br /><div><strong>BRANCH</strong><div><div>simplify_defaultValue</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D26766">https://phabricator.kde.org/D26766</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>autotests/kconfig_compiler/CMakeLists.txt<br />
autotests/kconfig_compiler/kconfigcompiler_test.cpp<br />
autotests/kconfig_compiler/test1.h.ref<br />
autotests/kconfig_compiler/test10.h.ref<br />
autotests/kconfig_compiler/test11.h.ref<br />
autotests/kconfig_compiler/test11a.h.ref<br />
autotests/kconfig_compiler/test12.h.ref<br />
autotests/kconfig_compiler/test13.h.ref<br />
autotests/kconfig_compiler/test9.h.ref<br />
autotests/kconfig_compiler/test_dpointer.cpp.ref<br />
autotests/kconfig_compiler/test_emptyentries.cpp.ref<br />
autotests/kconfig_compiler/test_emptyentries.h.ref<br />
autotests/kconfig_compiler/test_emptyentries.kcfg<br />
autotests/kconfig_compiler/test_emptyentries.kcfgc<br />
autotests/kconfig_compiler/test_emptyentries_main.cpp<br />
autotests/kconfig_compiler/test_translation_kde.h.ref<br />
autotests/kconfig_compiler/test_translation_kde_domain.h.ref<br />
autotests/kconfig_compiler/test_translation_qt.h.ref<br />
autotests/kconfigtest.h<br />
src/kconfig_compiler/CMakeLists.txt<br />
src/kconfig_compiler/KCFGXmlParser.cpp<br />
src/kconfig_compiler/KCFGXmlParser.h<br />
src/kconfig_compiler/KConfigCodeGeneratorBase.cpp<br />
src/kconfig_compiler/KConfigCodeGeneratorBase.h<br />
src/kconfig_compiler/KConfigCommonStructs.h<br />
src/kconfig_compiler/KConfigHeaderGenerator.cpp<br />
src/kconfig_compiler/KConfigHeaderGenerator.h<br />
src/kconfig_compiler/KConfigSourceGenerator.cpp<br />
src/kconfig_compiler/KConfigSourceGenerator.h<br />
src/kconfig_compiler/KConfigXTParameters.cpp<br />
src/kconfig_compiler/KConfigXTParameters.h<br />
src/kconfig_compiler/kconfig_compiler.cpp</div></div></div><br /><div><strong>To: </strong>tcanabrava<br /><strong>Cc: </strong>kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns<br /></div>