I wonder if we need this - we could assign the `NOT_FOUND` in the beginning of this method and then assign `currentDocFreq` as well as `currentTotalTermFreq` each time we update their local corresponding variables ie. as the last statement in the `for (Holder anEnum : enums)` loop. Same is true for the `text` and that way we can just trash everything below the for loop and return `found` I also think it's ok to just use `-1` as `NOT_FOUND`