Changeset 3638 for trunk/lang/octave

Timestamp:

Apr 26, 2013, 5:28:07 AM (11 years ago)

Author:

gah

Message:

fix ruby getstring for binary strings

File:

• trunk/lang/octave/src/rpLibGetString.cc (modified) (1 diff)

trunk/lang/octave/src/rpLibGetString.cc

@@ -41 +41 @@
     octave_value_list retval;
     int err = 1;
-    int nargin = args.length ();
-    std::string path = "";
-    int libHandle = 0;
-    RpLibrary* lib = NULL;
+    int argc;
     std::string retStr = "";
 
-    if (nargin == 2) {
-
-        if (    args(0).is_real_scalar () &&
-                args(1).is_string      ()   ) {
+    argc = args.length();
+    if (argc == 2) {
+        if ((args(0).is_real_scalar()) && (args(1).is_string())) {
+            std::string path;
+            int libHandle;
 
             libHandle = args(0).int_value ();
+
             path = args(1).string_value ();
-
             /* Call the C subroutine. */
             // path can be an empty string
-            if ( (libHandle >= 0) ) {
+            if (libHandle >= 0) {
+                RpLibrary* lib;
 
                 lib = (RpLibrary*) getObject_Void(libHandle);
-
-                if (lib) {
-                    retStr = lib->getString(path);
-                    err = 0;
+                if (lib != NULL) {
+                    retval(0) = lib->getString(path);
+                    retval(1) = 0;
+                    return retval;
                 }
-                else {
-                    // lib was null (not found in dictionary)
-                }
+            } else {
+                // invalid libHandle
+                _PRINT_USAGE(who.c_str());
             }
-            else {
-                // invalid libHandle
-                _PRINT_USAGE (who.c_str());
-            }
+        } else {
+            // wrong arg types
+            _PRINT_USAGE(who.c_str());
         }
-        else {
-            // wrong arg types
-            _PRINT_USAGE (who.c_str());
-        }
+    } else {
+        // wrong number of args.
+        _PRINT_USAGE(who.c_str());
     }
-    else {
-        // wrong number of args.
-        _PRINT_USAGE (who.c_str());
-    }
-
     retval(0) = retStr;
-    retval(1) = err;
+    retval(1) = 1;
     return retval;
 }