Replace various iterators with range-based loops (bug #63738) Replace various iterator-based loops with range-based loops where possible, for brevity. * libgui/src/documentation-bookmarks.cc: Use range-based loop * libgui/src/documentation.cc: Use range-based loop * libgui/src/files-dock-widget.cc: Use range-based loop * libgui/src/history-dock-widget.cc: Use range-based loop * libinterp/corefcn/gl-render.cc: Use range-based loop * libinterp/corefcn/gl2ps-print.cc: Use range-based loop * libinterp/corefcn/graphics.cc: Use range-based loop * libinterp/corefcn/sysdep.cc: Use range-based loop * libinterp/dldfcn/__init_fltk__.cc: Use range-based loop * libinterp/octave-value/cdef-class.cc: Use range-based loop * libinterp/octave-value/cdef-package.cc: Use range-based loop * libgui/src/find-files-model.cc: Add FIXME to note sorted insertion. * src/octave-svgconvert.cc: Use range-based loop, add FIXME to note duplicate code, eliminate loop with call to 2-argument constructor.

--- a/libinterp/corefcn/sysdep.cc	Wed Feb 01 16:10:44 2023 -0500
+++ b/libinterp/corefcn/sysdep.cc	Thu Feb 02 11:41:24 2023 -0500
@@ -1134,10 +1134,9 @@
         error ("winqueryreg: error %ld reading names from registry", retval);
 
       Cell fieldnames (dim_vector (1, fields.size ()));
-      std::size_t i;
-      std::list<std::string>::const_iterator it;
-      for (i = 0, it = fields.begin (); it != fields.end (); ++it, ++i)
-        fieldnames(i) = *it;
+      std::size_t i = 0;
+      for (const auto& it : fields)
+        fieldnames(i++) = it;
 
       return ovl (fieldnames);
     }