ervin updated this revision to Diff 67516. ervin added a comment.
Address Friedrich's comments, I went for hiding the feature behind a flag after all, it was just less complexity added to the compiler in the end REPOSITORY R237 KConfig CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D24490?vs=67490&id=67516 REVISION DETAIL https://phabricator.kde.org/D24490 AFFECTED FILES autotests/kconfig_compiler/test1.cpp.ref autotests/kconfig_compiler/test1.h.ref autotests/kconfig_compiler/test1.kcfgc autotests/kconfig_compiler/test10.cpp.ref autotests/kconfig_compiler/test11.cpp.ref autotests/kconfig_compiler/test11.h.ref autotests/kconfig_compiler/test11.kcfgc autotests/kconfig_compiler/test11a.cpp.ref autotests/kconfig_compiler/test12.cpp.ref autotests/kconfig_compiler/test13.cpp.ref autotests/kconfig_compiler/test2.cpp.ref autotests/kconfig_compiler/test3.cpp.ref autotests/kconfig_compiler/test3a.cpp.ref autotests/kconfig_compiler/test4.cpp.ref autotests/kconfig_compiler/test5.cpp.ref autotests/kconfig_compiler/test5.h.ref autotests/kconfig_compiler/test5.kcfgc autotests/kconfig_compiler/test6.cpp.ref autotests/kconfig_compiler/test7.cpp.ref autotests/kconfig_compiler/test8a.cpp.ref autotests/kconfig_compiler/test8a.h.ref autotests/kconfig_compiler/test8a.kcfgc autotests/kconfig_compiler/test8b.cpp.ref autotests/kconfig_compiler/test8c.cpp.ref autotests/kconfig_compiler/test8c.h.ref autotests/kconfig_compiler/test8c.kcfgc autotests/kconfig_compiler/test9.cpp.ref autotests/kconfig_compiler/test_dpointer.cpp.ref autotests/kconfig_compiler/test_notifiers.cpp.ref autotests/kconfig_compiler/test_qdebugcategory.cpp.ref autotests/kconfig_compiler/test_signal.cpp.ref autotests/kconfig_compiler/test_translation_kde.cpp.ref autotests/kconfig_compiler/test_translation_kde_domain.cpp.ref autotests/kconfig_compiler/test_translation_qt.cpp.ref src/kconfig_compiler/kconfig_compiler.cpp To: ervin, #plasma, #frameworks, dfaure, mart, apol Cc: kossebau, apol, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns