[ 
https://issues.apache.org/jira/browse/THRIFT-572?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12750157#action_12750157
 ] 

Bryan Duxbury commented on THRIFT-572:
--------------------------------------

Hey Brett, can you attach this fix as a patch relative to the root of the svn 
(or git) project? That allows you to check the apache license box.

> fix RSTRING for 1.9 compatibility
> ---------------------------------
>
>                 Key: THRIFT-572
>                 URL: https://issues.apache.org/jira/browse/THRIFT-572
>             Project: Thrift
>          Issue Type: Bug
>          Components: Library (Ruby)
>         Environment: osx 10.6/intel
> ruby 1.9.1
> thrift HEAD
>            Reporter: Brett Eisenberg
>
> @@ -54,7 +54,7 @@ VALUE rb_thrift_memory_buffer_read(VALUE self, VALUE 
> length_value) {
>      index = 0;
>    }
> -  if (RSTRING(data)->len < length) {
> +  if (RSTRING_LEN(data) < length) {
>      rb_raise(rb_eEOFError, "Not enough bytes remain in memory buffer");
>    }
>  

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to