[
https://issues.apache.org/jira/browse/THRIFT-677?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12800632#action_12800632
]
Enis Soztutar commented on THRIFT-677:
--------------------------------------
bq. When you say that the class is "still used by legacy code", is there some
reason this legacy code can't be regenerated?
For our use cases, we have not updated thrift for a while. There may be others
like this.
bq. In all honesty, I'd forgotten that this class was even still in the
codebase, and I think the correct thing for us to do is to remove it.
Agreed that if this class will no longer be used it is better to remove it
instead of applying this patch.
> 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.
