Yonik Seeley wrote: > The else clause in SegmentTermPositions.readDeltaPosition() is > redundant and could be removed, yes? > It's a pretty minor improvement, but this is very inner-loop stuff. > > -Yonik >
Thanks, Yonik, you're right. We can safely remove those two lines. TermPositions#seek() resets the two values. And "currentFieldStoresPayloads" doesn't change unless seek() is called. All test cases still pass after removing the else clause. I'll commit this small change (I don't think we need to open a Jira issue). -Michael --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
