You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@qpid.apache.org by tr...@apache.org on 2009/08/12 16:44:22 UTC

svn commit: r803541 - in /qpid/trunk/qpid/cpp/bindings/qmf/ruby: qmf.rb ruby.i

Author: tross
Date: Wed Aug 12 14:44:22 2009
New Revision: 803541

URL: http://svn.apache.org/viewvc?rev=803541&view=rev
Log:
Fixed a bug where Ruby/C number conversions failed on 32-bit architectures.

Modified:
    qpid/trunk/qpid/cpp/bindings/qmf/ruby/qmf.rb
    qpid/trunk/qpid/cpp/bindings/qmf/ruby/ruby.i

Modified: qpid/trunk/qpid/cpp/bindings/qmf/ruby/qmf.rb
URL: http://svn.apache.org/viewvc/qpid/trunk/qpid/cpp/bindings/qmf/ruby/qmf.rb?rev=803541&r1=803540&r2=803541&view=diff
==============================================================================
--- qpid/trunk/qpid/cpp/bindings/qmf/ruby/qmf.rb (original)
+++ qpid/trunk/qpid/cpp/bindings/qmf/ruby/qmf.rb Wed Aug 12 14:44:22 2009
@@ -485,21 +485,21 @@
         end
       end
 
-      def set_attr(name, value)
+      def set_attr(name, v)
         val = value(name)
         case val.getType
-        when TYPE_UINT8, TYPE_UINT16, TYPE_UINT32 then val.setUint(value)
-        when TYPE_UINT64 then val.setUint64(value)
-        when TYPE_SSTR, TYPE_LSTR then value ? val.setString(value) : val.setString('')
-        when TYPE_ABSTIME then val.setInt64(value)
-        when TYPE_DELTATIME then val.setUint64(value)
-        when TYPE_REF then val.setObjectId(value.impl)
-        when TYPE_BOOL then value ? val.setBool(value) : val.setBool(0)
-        when TYPE_FLOAT then val.setFloat(value)
-        when TYPE_DOUBLE then val.setDouble(value)
-        when TYPE_UUID then val.setUuid(value)
-        when TYPE_INT8, TYPE_INT16, TYPE_INT32 then val.setInt(value)
-        when TYPE_INT64 then val.setInt64(value)
+        when TYPE_UINT8, TYPE_UINT16, TYPE_UINT32 then val.setUint(v)
+        when TYPE_UINT64 then val.setUint64(v)
+        when TYPE_SSTR, TYPE_LSTR then v ? val.setString(v) : val.setString('')
+        when TYPE_ABSTIME then val.setInt64(v)
+        when TYPE_DELTATIME then val.setUint64(v)
+        when TYPE_REF then val.setObjectId(v.impl)
+        when TYPE_BOOL then v ? val.setBool(v) : val.setBool(0)
+        when TYPE_FLOAT then val.setFloat(v)
+        when TYPE_DOUBLE then val.setDouble(v)
+        when TYPE_UUID then val.setUuid(v)
+        when TYPE_INT8, TYPE_INT16, TYPE_INT32 then val.setInt(v)
+        when TYPE_INT64 then val.setInt64(v)
         when TYPE_MAP
         when TYPE_OBJECT
         when TYPE_LIST

Modified: qpid/trunk/qpid/cpp/bindings/qmf/ruby/ruby.i
URL: http://svn.apache.org/viewvc/qpid/trunk/qpid/cpp/bindings/qmf/ruby/ruby.i?rev=803541&r1=803540&r2=803541&view=diff
==============================================================================
--- qpid/trunk/qpid/cpp/bindings/qmf/ruby/ruby.i (original)
+++ qpid/trunk/qpid/cpp/bindings/qmf/ruby/ruby.i Wed Aug 12 14:44:22 2009
@@ -33,7 +33,7 @@
 
 %typemap (in) uint16_t
 {
-    $1 = FIX2UINT ($input);
+    $1 = NUM2UINT ($input);
 }
 
 %typemap (out) uint16_t
@@ -43,7 +43,7 @@
 
 %typemap (in) uint32_t
 {
-    $1 = FIX2UINT ($input);
+    $1 = NUM2UINT ($input);
 }
 
 %typemap (out) uint32_t
@@ -57,7 +57,7 @@
 
 %typemap (in) uint64_t
 {
-    $1 = FIX2ULONG ($input);
+    $1 = NUM2ULONG ($input);
 }
 
 %typemap (out) uint64_t



---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:commits-subscribe@qpid.apache.org