[ 
https://issues.apache.org/jira/browse/THRIFT-677?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Enis Soztutar updated THRIFT-677:
---------------------------------

    Attachment: intrangeset_v1.patch

I am attaching a patch, complete with a unit test. The patch also enables empty 
intrangesets. 

BTW, why thrift does not use a well established unit testing framework? 

> IntRangeSet.java ignores first element in the constructor array
> ---------------------------------------------------------------
>
>                 Key: THRIFT-677
>                 URL: https://issues.apache.org/jira/browse/THRIFT-677
>             Project: Thrift
>          Issue Type: Bug
>          Components: Library (Java)
>            Reporter: Enis Soztutar
>         Attachments: intrangeset_v1.patch
>
>
> Although IntRangeSet is no more used since generation of enums using native 
> java enums, the class is still used by legacy code. 
> The constructor for IntRangeSet does not put the first element in the given 
> array to the realSet, which results in incorrect behavior. 

-- 
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