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/gl2ps-print.cc	Wed Feb 01 16:10:44 2023 -0500
+++ b/libinterp/corefcn/gl2ps-print.cc	Thu Feb 02 11:41:24 2023 -0500
@@ -1010,8 +1010,8 @@
       // provide an x coordinate for each character in the string
       os << "x=\"";
       std::vector<double> xdata = p->get_xdata ();
-      for (auto q = xdata.begin (); q != xdata.end (); q++)
-        os << (*q) << " ";
+      for (const auto& q : xdata)
+        os << q << " ";
       os << '"';
 
       os << '>';
@@ -1022,10 +1022,10 @@
       else
         {
           const std::string str = p->get_string ();
-          for (auto q = str.begin (); q != str.end (); q++)
+          for (const auto& q : str)
             {
               std::stringstream chr;
-              chr << *q;
+              chr << q;
               if (chr.str () == "\"")
                 os << "&quot;";
               else if (chr.str () == "'")