[ https://issues.apache.org/jira/browse/LUCENE-1616?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12703085#action_12703085 ]
Eks Dev commented on LUCENE-1616: --------------------------------- I am ok with both options, removing separate looks a bit better for me as it forces users to think "attomic" about offset <=> {start, end}. If you separate start and end offset too far in your code, probability that you do not see mistake somewhere is higher compared to the case where you manage start and end on your own in these cases (this is then rather "explicit" in you code)... But that is all really something we should not think too much about it :) We make no mistakes eather way I can provide new patch, if needed. > add one setter for start and end offset to OffsetAttribute > ---------------------------------------------------------- > > Key: LUCENE-1616 > URL: https://issues.apache.org/jira/browse/LUCENE-1616 > Project: Lucene - Java > Issue Type: Improvement > Components: Analysis > Reporter: Eks Dev > Priority: Trivial > Fix For: 2.9 > > Attachments: LUCENE-1616.patch > > > add OffsetAttribute. setOffset(startOffset, endOffset); > trivial change, no JUnit needed > Changed CharTokenizer to use it -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. --------------------------------------------------------------------- To unsubscribe, e-mail: java-dev-unsubscr...@lucene.apache.org For additional commands, e-mail: java-dev-h...@lucene.apache.org