Minor code optimisation

- Use more range-for loops where appropriate
- Use auto instead of the usually-long iterator type names
- Use cbegin/cend(), to match the std:: containers, less confusion
- Use qDeleteAll instead of a for loop
- Make a QRE with a long-ish pattern static

NO_CHANGELOG

1 files changed, 1 insertions, 4 deletions

diff --git a/src/core/kcoreconfigskeleton_p.h b/src/core/kcoreconfigskeleton_p.h
index d854cc43..a7417430 100644
--- a/src/core/kcoreconfigskeleton_p.h
+++ b/src/core/kcoreconfigskeleton_p.h
@@ -21,10 +21,7 @@ public:
     }
     ~KCoreConfigSkeletonPrivate()
     {
-        KConfigSkeletonItem::List::ConstIterator it;
-        for (it = mItems.constBegin(); it != mItems.constEnd(); ++it) {
-            delete *it;
-        }
+        qDeleteAll(mItems);
     }
     QString mCurrentGroup;