You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@thrift.apache.org by br...@apache.org on 2009/03/20 03:21:06 UTC
svn commit: r756308 -
/incubator/thrift/trunk/lib/rb/ext/binary_protocol_accelerated.c
Author: bryanduxbury
Date: Fri Mar 20 02:21:05 2009
New Revision: 756308
URL: http://svn.apache.org/viewvc?rev=756308&view=rev
Log:
THRIFT-313. rb: BinaryProtocolAccelerated and BinaryProtocol don't produce the same bytes when writes aren't strict
A Fixnum was not properly being converted to its native value before being written. This patch rectifies that problem.
Modified:
incubator/thrift/trunk/lib/rb/ext/binary_protocol_accelerated.c
Modified: incubator/thrift/trunk/lib/rb/ext/binary_protocol_accelerated.c
URL: http://svn.apache.org/viewvc/incubator/thrift/trunk/lib/rb/ext/binary_protocol_accelerated.c?rev=756308&r1=756307&r2=756308&view=diff
==============================================================================
--- incubator/thrift/trunk/lib/rb/ext/binary_protocol_accelerated.c (original)
+++ incubator/thrift/trunk/lib/rb/ext/binary_protocol_accelerated.c Fri Mar 20 02:21:05 2009
@@ -103,7 +103,7 @@
write_i32_direct(trans, FIX2INT(seqid));
} else {
write_string_direct(trans, name);
- write_byte_direct(trans, type);
+ write_byte_direct(trans, FIX2INT(type));
write_i32_direct(trans, FIX2INT(seqid));
}