qpid-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Gordon Sim <g...@redhat.com>
Subject Re: RFQ 0.24: Two JIRAs
Date Mon, 12 Aug 2013 12:25:51 GMT
On 08/12/2013 11:31 AM, Gordon Sim wrote:
> On 08/12/2013 11:26 AM, Gordon Sim wrote:
>>> Modified: qpid/trunk/qpid/cpp/include/qpid/swig_ruby_typemaps.i
>>> URL:
>>> http://svn.apache.org/viewvc/qpid/trunk/qpid/cpp/include/qpid/swig_ruby_typemaps.i?rev=1512482&r1=1512481&r2=1512482&view=diff
>>>
>>> ==============================================================================
>>>
>>> --- qpid/trunk/qpid/cpp/include/qpid/swig_ruby_typemaps.i (original)
>>> +++ qpid/trunk/qpid/cpp/include/qpid/swig_ruby_typemaps.i Fri Aug  9
>>> 20:28:59 2013
>>> @@ -27,9 +27,16 @@
>>>       void RbToList(VALUE, qpid::types::Variant::List*);
>>>
>>>       qpid::types::Variant RbToVariant(VALUE value) {
>>> +      printf("RbToVariant is of type %d\n", TYPE(value));
>>>           switch (TYPE(value)) {
>>>           case T_FLOAT:   return qpid::types::Variant(NUM2DBL(value));
>>> -        case T_STRING:  return
>>> qpid::types::Variant(StringValuePtr(value));
>>> +        case T_STRING: {
>>> +          qpid::types::Variant v = qpid::types::Variant();
>>> +          v = StringValuePtr(value));

The above looks like a typo (extra bracket?) and might explain:

> [ 96%] Building CXX object bindings/qpid/ruby/CMakeFiles/cqpid_ruby.dir/rubyRUBY_wrap.o
> <http://mrg23.lab.bos.redhat.com:8080/job/qpid-cpp-trunk-cmake/label=master/ws/qpid/cpp/cpp-vpath/bindings/qpid/ruby/rubyRUBY_wrap.cxx>:
In function ‘qpid::types::Variant RbToVariant(VALUE)’:
> <http://mrg23.lab.bos.redhat.com:8080/job/qpid-cpp-trunk-cmake/label=master/ws/qpid/cpp/cpp-vpath/bindings/qpid/ruby/rubyRUBY_wrap.cxx>:2179:
error: expected ‘;’ before ‘)’ token
> make[2]: *** [bindings/qpid/ruby/CMakeFiles/cqpid_ruby.dir/rubyRUBY_wrap.o] Error 1
> make[1]: *** [bindings/qpid/ruby/CMakeFiles/cqpid_ruby.dir/all] Error 2


>>> +          v.setEncoding("utf8");
>>> +          return v;
>>> +        }
>>> +          break;
>>>           case T_FIXNUM:  return qpid::types::Variant((int64_t)
>>> FIX2LONG(value));
>>>           case T_BIGNUM:  return qpid::types::Variant((int64_t)
>>> NUM2LL(value));
>>>           case T_TRUE:    return qpid::types::Variant(true);





---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org


Mime
View raw message