Modernize sequence conversions

- Use correct size types
- Use const iterators
- Avoid repeated invocation of end()/size()

Task-number: PYSIDE-1438
Change-Id: I7d30a5c87d0867400134be96ee61ff1ab6129435
Reviewed-by: Christian Tismer <tismer@stackless.com>

1 files changed, 7 insertions, 7 deletions

diff --git a/sources/shiboken6/tests/samplebinding/typesystem_sample.xml b/sources/shiboken6/tests/samplebinding/typesystem_sample.xml
index 20fa61334..a8b7b3b11 100644
--- a/sources/shiboken6/tests/samplebinding/typesystem_sample.xml
+++ b/sources/shiboken6/tests/samplebinding/typesystem_sample.xml
@@ -388,9 +388,9 @@
         </conversion-rule>
     </container-type>
     <template name="cpplist_to_pylist_convertion">
-    PyObject* %out = PyList_New((int) %in.size());
-    %INTYPE::const_iterator it = %in.begin();
-    for (int idx = 0; it != %in.end(); ++it, ++idx) {
+    PyObject *%out = PyList_New(Py_ssize_t(%in.size()));
+    Py_ssize_t idx = 0;
+    for (auto it = %in.cbegin(), end = %in.cend(); it != end; ++it, ++idx) {
         %INTYPE_0 cppItem(*it);
         PyList_SET_ITEM(%out, idx, %CONVERTTOPYTHON[%INTYPE_0](cppItem));
     }
@@ -398,7 +398,8 @@
     </template>
     <template name="pyseq_to_cpplist_convertion">
     Shiboken::AutoDecRef seq(PySequence_Fast(%in, 0));
-    for (int i = 0; i &lt; PySequence_Fast_GET_SIZE(seq.object()); i++) {
+    const Py_ssize_t size = PySequence_Fast_GET_SIZE(seq.object());
+    for (Py_ssize_t i = 0; i &lt; size; ++i) {
         PyObject* pyItem = PySequence_Fast_GET_ITEM(seq.object(), i);
         %OUTTYPE_0 cppItem = %CONVERTTOCPP[%OUTTYPE_0](pyItem);
         %out.push_back(cppItem);
@@ -435,8 +436,7 @@
         <conversion-rule>
             <native-to-target>
             PyObject* %out = PyDict_New();
-            %INTYPE::const_iterator it = %in.begin();
-            for (; it != %in.end(); ++it) {
+            for (auto it = %in.cbegin(), end = %in.cend(); it != end; ++it) {
                 %INTYPE_0 key = it->first;
                 %INTYPE_1 value = it->second;
                 PyDict_SetItem(%out,
@@ -453,7 +453,7 @@
                 while (PyDict_Next(%in, &amp;pos, &amp;key, &amp;value)) {
                     %OUTTYPE_0 cppKey = %CONVERTTOCPP[%OUTTYPE_0](key);
                     %OUTTYPE_1 cppValue = %CONVERTTOCPP[%OUTTYPE_1](value);
-                    %out.insert(%OUTTYPE::value_type(cppKey, cppValue));
+                    %out.insert({cppKey, cppValue});
                 }
                 </add-conversion>
             </target-to-native>