Eradicate Java-style iterators and mark the module free of them

Java-style iterators are scheduled for deprecation, or at the very
least banned from use in Qt's own implementation.

Change-Id: I3b65526968551baf48d1acb1e7184a3800092b56
Reviewed-by: Mårten Nordheim <[email protected]>

Author: marc-kdab

.qmake.conf

@@ -1,3 +1,5 @@
 load(qt_build_config)
 
+DEFINES += QT_NO_JAVA_STYLE_ITERATORS
+
 MODULE_VERSION = 5.14.0

src/plugins/platforms/webgl/qwebglcontext.cpp

@@ -228,9 +228,8 @@ static int bufferSize(GLsizei count, GLint elemsPerVertex, GLenum type, GLsizei
 static void setVertexAttribs(QWebGLFunctionCall *event, GLsizei count)
 {
     event->addInt(currentContextData()->vertexAttribPointers.count());
-    QHashIterator<GLuint, ContextData::VertexAttrib> it(currentContextData()->vertexAttribPointers);
-    while (it.hasNext()) {
-        it.next();
+    const auto &vertexAttribPointers = currentContextData()->vertexAttribPointers;
+    for (auto it = vertexAttribPointers.cbegin(), end = vertexAttribPointers.cend(); it != end; ++it) {
         const ContextData::VertexAttrib &va(it.value());
         if (va.arrayBufferBinding == 0 && va.enabled) {
             int len = bufferSize(count, va.size, va.type, va.stride);