changeset 23911:212af6615820 stable

don't return cs-list from group and password functions * getgrent.cc (mk_gr_map): Undo change made in f1b2a2dbc0e1. This function returns an octave_value, so it is not correct to assign an octave_value_list to retval. * getpwent.cc (mk_pw_map): Likewise.
author John W. Eaton <jwe@octave.org>
date Mon, 14 Aug 2017 08:39:17 -0400
parents 437ba51e8bcf
children 21c2fabd6ed2 520c37805969
files libinterp/corefcn/getgrent.cc libinterp/corefcn/getpwent.cc
diffstat 2 files changed, 4 insertions(+), 10 deletions(-) [+]
line wrap: on
line diff
--- a/libinterp/corefcn/getgrent.cc	Sat Aug 12 02:23:56 2017 +0200
+++ b/libinterp/corefcn/getgrent.cc	Mon Aug 14 08:39:17 2017 -0400
@@ -43,8 +43,6 @@
 static octave_value
 mk_gr_map (const octave::sys::group& gr)
 {
-  octave_value retval;
-
   if (gr)
     {
       octave_scalar_map m;
@@ -54,12 +52,10 @@
       m.assign ("gid", static_cast<double> (gr.gid ()));
       m.assign ("mem", octave_value (gr.mem ()));
 
-      retval = ovl (m);
+      return octave_value (m);
     }
   else
-    retval = ovl (0);
-
-  return retval;
+    return octave_value (0);
 }
 
 DEFUN (getgrent, args, ,
--- a/libinterp/corefcn/getpwent.cc	Sat Aug 12 02:23:56 2017 +0200
+++ b/libinterp/corefcn/getpwent.cc	Mon Aug 14 08:39:17 2017 -0400
@@ -57,12 +57,10 @@
       m.assign ("dir", pw.dir ());
       m.assign ("shell", pw.shell ());
 
-      retval = ovl (m);
+      return octave_value (m);
     }
   else
-    retval = ovl (0);
-
-  return retval;
+    return octave_value (0);
 }
 
 DEFUN (getpwent, args, ,