aboutsummaryrefslogtreecommitdiff
path: root/autotests/kconfig_compiler/kconfigcompiler_test.cpp
diff options
context:
space:
mode:
authorAhmad Samir <a.samirh78@gmail.com>2021-02-03 15:28:38 +0200
committerAhmad Samir <a.samirh78@gmail.com>2021-02-06 22:11:41 +0200
commitdc878289c01011c59615746655b4e78d4bc90b91 (patch)
treea5c547a6d05e46efac6e17725d75343b40e78440 /autotests/kconfig_compiler/kconfigcompiler_test.cpp
parentbb16fda4e5f7caa9e892540ec69a202cec9eb16f (diff)
downloadkconfig-dc878289c01011c59615746655b4e78d4bc90b91.tar.gz
kconfig-dc878289c01011c59615746655b4e78d4bc90b91.tar.bz2
Minor code optimisations
- Use a global var for a QString that's used many times - Break up long-all-cap variable names, it makes it harder to read (and I've fixed one typo in one of those ALLCAPS) - Fix some clazy warnings, make global QString objects in unit tests static (so that the compiler doesn't create symbols for them, it doesn't matter in a unit test but KF code acts as a reference sometimes that others copy from, tip from dfaure) - Add TODO note about changing kconfig_compiler to generate C++ code that uses multi-arg QString::arg(QString, QString, QString) instead of QString::arg().arg() - More const; more QString::at() instead of operator[] where appropriate - Use a ternary where it makes the code more readable (and uses less lines :)) NO_CHANGELOG
Diffstat (limited to 'autotests/kconfig_compiler/kconfigcompiler_test.cpp')
0 files changed, 0 insertions, 0 deletions