diff options
author | Ahmad Samir <a.samirh78@gmail.com> | 2021-02-03 15:28:38 +0200 |
---|---|---|
committer | Ahmad Samir <a.samirh78@gmail.com> | 2021-02-06 22:11:41 +0200 |
commit | dc878289c01011c59615746655b4e78d4bc90b91 (patch) | |
tree | a5c547a6d05e46efac6e17725d75343b40e78440 /autotests/kconfig_compiler/kconfigcompiler_test.cpp | |
parent | bb16fda4e5f7caa9e892540ec69a202cec9eb16f (diff) | |
download | kconfig-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