nikcio commented on code in PR #654:
URL: https://github.com/apache/lucenenet/pull/654#discussion_r995682325
##########
src/Lucene.Net.Analysis.Common/Analysis/Fr/FrenchStemmer.cs:
##########
@@ -483,21 +483,21 @@ private void DeleteButSuffixFromElseReplace(string
source, string[] search, stri
{
if (source.EndsWith(prefix + search[i],
StringComparison.Ordinal))
{
- sb.Remove(sb.Length - (prefix.Length +
search[i].Length), sb.Length - sb.Length - (prefix.Length + search[i].Length));
+ sb.Remove(sb.Length - (prefix.Length +
search[i].Length), sb.Length);
Review Comment:
I tried to look through the source code with some Regex in hopes of finding
other places with the same bug but I didn't find any so maybe it's already been
fixed in other places.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
