[
https://issues.apache.org/jira/browse/THRIFT-677?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12800281#action_12800281
]
Bryan Duxbury commented on THRIFT-677:
--------------------------------------
When you say that the class is "still used by legacy code", is there some
reason this legacy code can't be regenerated? 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.
We don't use a unit testing framework just because no one has taken the time to
convert our existing suite. I'd love to tackle that at some point, but haven't
had the time.
> 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.
