changeset 21862:5c67b16acc4a

maint: Use C++ 'auto' keyword in some instances in liboctave. * cmd-edit.cc, cmd-edit.h, oct-shlib.cc, singleton-cleanup.cc: Use C++ 'auto' keyword in some instances in liboctave.
author Rik <rik@octave.org>
date Sat, 11 Jun 2016 15:32:31 -0700
parents a55f1fa2c94a
children a25110491607
files liboctave/util/cmd-edit.cc liboctave/util/cmd-edit.h liboctave/util/oct-shlib.cc liboctave/util/singleton-cleanup.cc
diffstat 4 files changed, 14 insertions(+), 27 deletions(-) [+]
line wrap: on
line diff
--- a/liboctave/util/cmd-edit.cc	Sat Jun 11 13:13:03 2016 -0700
+++ b/liboctave/util/cmd-edit.cc	Sat Jun 11 15:32:31 2016 -0700
@@ -1090,10 +1090,9 @@
   int
   command_editor::startup_handler (void)
   {
-    for (startup_hook_set_iterator p = startup_hook_set.begin ();
-         p != startup_hook_set.end (); p++)
+    for (auto fcnptr : startup_hook_set)
       {
-        startup_hook_fcn f = *p;
+        startup_hook_fcn f = *fcnptr;
 
         if (f)
           f ();
@@ -1105,10 +1104,9 @@
   int
   command_editor::pre_input_handler (void)
   {
-    for (pre_input_hook_set_iterator p = pre_input_hook_set.begin ();
-         p != pre_input_hook_set.end (); p++)
+    for (auto fcnptr : pre_input_hook_set)
       {
-        pre_input_hook_fcn f = *p;
+        pre_input_hook_fcn f = *fcnptr;
 
         if (f)
           f ();
@@ -1126,10 +1124,9 @@
 
     event_hook_lock.unlock ();
 
-    for (event_hook_set_iterator p = hook_set.begin ();
-         p != hook_set.end (); p++)
+    for (auto fcnptr : hook_set)
       {
-        event_hook_fcn f = *p;
+        event_hook_fcn f = *fcnptr;
 
         if (f)
           f ();
@@ -1482,7 +1479,7 @@
   {
     if (instance_ok ())
       {
-        startup_hook_set_iterator p = startup_hook_set.find (f);
+        auto p = startup_hook_set.find (f);
 
         if (p != startup_hook_set.end ())
           startup_hook_set.erase (p);
@@ -1508,7 +1505,7 @@
   {
     if (instance_ok ())
       {
-        pre_input_hook_set_iterator p = pre_input_hook_set.find (f);
+        auto p = pre_input_hook_set.find (f);
 
         if (p != pre_input_hook_set.end ())
           pre_input_hook_set.erase (p);
@@ -1538,7 +1535,7 @@
 
     if (instance_ok ())
       {
-        event_hook_set_iterator p = event_hook_set.find (f);
+        auto p = event_hook_set.find (f);
 
         if (p != event_hook_set.end ())
           event_hook_set.erase (p);
--- a/liboctave/util/cmd-edit.h	Sat Jun 11 13:13:03 2016 -0700
+++ b/liboctave/util/cmd-edit.h	Sat Jun 11 15:32:31 2016 -0700
@@ -213,15 +213,6 @@
 
     static std::set<event_hook_fcn> event_hook_set;
 
-    typedef std::set<startup_hook_fcn>::iterator startup_hook_set_iterator;
-    typedef std::set<startup_hook_fcn>::const_iterator startup_hook_set_const_iterator;
-
-    typedef std::set<pre_input_hook_fcn>::iterator pre_input_hook_set_iterator;
-    typedef std::set<pre_input_hook_fcn>::const_iterator pre_input_hook_set_const_iterator;
-
-    typedef std::set<event_hook_fcn>::iterator event_hook_set_iterator;
-    typedef std::set<event_hook_fcn>::const_iterator event_hook_set_const_iterator;
-
     // The real thing.
     static command_editor *instance;
 
--- a/liboctave/util/oct-shlib.cc	Sat Jun 11 13:13:03 2016 -0700
+++ b/liboctave/util/oct-shlib.cc	Sat Jun 11 15:32:31 2016 -0700
@@ -116,7 +116,7 @@
   void
   dynamic_library::dynlib_rep::add_fcn_name (const std::string& name)
   {
-    fcn_names_iterator p = fcn_names.find (name);
+    auto p = fcn_names.find (name);
 
     if (p == fcn_names.end ())
       fcn_names[name] = 1;
@@ -129,7 +129,7 @@
   {
     bool retval = false;
 
-    fcn_names_iterator p = fcn_names.find (fcn_name);
+    auto p = fcn_names.find (fcn_name);
 
     if (p != fcn_names.end () && --(p->second) == 0)
       {
@@ -143,7 +143,7 @@
   void
   dynamic_library::dynlib_rep::do_close_hook (dynamic_library::close_hook cl_hook)
   {
-    for (fcn_names_iterator p = fcn_names.begin (); p != fcn_names.end (); p++)
+    for (auto p = fcn_names.begin (); p != fcn_names.end (); p++)
       cl_hook (p->first);
 
     fcn_names.clear ();
--- a/liboctave/util/singleton-cleanup.cc	Sat Jun 11 13:13:03 2016 -0700
+++ b/liboctave/util/singleton-cleanup.cc	Sat Jun 11 15:32:31 2016 -0700
@@ -31,10 +31,9 @@
 
 singleton_cleanup_list::~singleton_cleanup_list (void)
 {
-  for (std::set<fptr>::iterator p = fcn_list.begin ();
-       p != fcn_list.end (); p++)
+  for (auto fcnptr : fcn_list)
     {
-      fptr fcn = *p;
+      fptr fcn = *fcnptr;
 
       fcn ();
     }