paulirwin commented on code in PR #1049:
URL: https://github.com/apache/lucenenet/pull/1049#discussion_r1859473842
##########
src/Lucene.Net.Tests.Analysis.Common/Analysis/Position/PositionFilterTest.cs:
##########
@@ -105,7 +107,7 @@ public virtual void TestReset()
/// <summary>
/// Tests ShingleFilter up to six shingles against six terms.
/// Tests PositionFilter setting all but the first positionIncrement
to zero. </summary> </exception>
- /// <exception cref="java.io.IOException"> <seealso cref=
Token#next(Token) </seealso>
+ /// <exception cref="IOException"> <seealso cref= Token#next(Token)
</seealso>
Review Comment:
I am not sure what this is supposed to be pointing to, seems like an
outdated reference. Token does not have a `Next(Token)` method in either .NET
or Java. It looks from the upstream CHANGES doc that this hasn't been a thing
since Lucene 2.9. PositionFilter also doesn't seem to throw IOException itself.
Going to just remove this for now.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@lucenenet.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
