[lucenenet] branch master updated: fix: Order of precedence for translation of Remove() method args in FrenchStemmer.cs (#654)

[nightowl888]

This is an automated email from the ASF dual-hosted git repository.

nightowl888 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/lucenenet.git


The following commit(s) were added to refs/heads/master by this push:
     new 3dca1619c fix: Order of precedence for translation of Remove() method 
args in FrenchStemmer.cs  (#654)
3dca1619c is described below

commit 3dca1619cc92d91ec2513372a838f4e7b9b86abe
Author: Nikolaj Brask-Nielsen <nikolajg...@gmail.com>
AuthorDate: Fri Oct 14 20:36:29 2022 +0200

    fix: Order of precedence for translation of Remove() method args in 
FrenchStemmer.cs  (#654)
    
    * fix: Aligned DeleteButSuffixFromElseReplace with Java code
    
    [File: FrenchStemmer.cs]
    
    Java reference: 
https://github.com/apache/lucene/blob/f01152a5909fa6059f4f1d4aeb4e14968ef1d8c2/lucene/analysis/common/src/java/org/apache/lucene/analysis/fr/FrenchStemmer.java#L450-L477
    
    * fix: Add parentheses to string builder remove arguments
---
 src/Lucene.Net.Analysis.Common/Analysis/Fr/FrenchStemmer.cs | 10 +++++-----
 1 file changed, 5 insertions(+), 5 deletions(-)

diff --git a/src/Lucene.Net.Analysis.Common/Analysis/Fr/FrenchStemmer.cs 
b/src/Lucene.Net.Analysis.Common/Analysis/Fr/FrenchStemmer.cs
index 1dcae7968..6a947a133 100644
--- a/src/Lucene.Net.Analysis.Common/Analysis/Fr/FrenchStemmer.cs
+++ b/src/Lucene.Net.Analysis.Common/Analysis/Fr/FrenchStemmer.cs
@@ -1,4 +1,4 @@
-// Lucene version compatibility level 4.8.1
+// Lucene version compatibility level 4.8.1
 using System;
 using System.Globalization;
 using System.Text;
@@ -483,21 +483,21 @@ namespace Lucene.Net.Analysis.Fr
                 {
                     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 - (sb.Length - (prefix.Length + 
search[i].Length)));
                         modified = true;
                         SetStrings();
                         break;
                     }
                     else if (from != null && from.EndsWith(prefix + search[i], 
StringComparison.Ordinal))
                     {
-                        sb.Remove(sb.Length - (prefix.Length + 
search[i].Length), sb.Length - sb.Length - (prefix.Length + 
search[i].Length)).Insert(sb.Length - (prefix.Length + search[i].Length), 
replace);
+                        sb.Remove(sb.Length - (prefix.Length + 
search[i].Length), sb.Length - (sb.Length - (prefix.Length + 
search[i].Length))).Insert(sb.Length - (prefix.Length + search[i].Length), 
replace);
                         modified = true;
                         SetStrings();
                         break;
                     }
                     else if (without && source.EndsWith(search[i], 
StringComparison.Ordinal))
                     {
-                        sb.Remove(sb.Length - search[i].Length, sb.Length - 
sb.Length - search[i].Length);
+                        sb.Remove(sb.Length - search[i].Length, sb.Length - 
(sb.Length - search[i].Length));
                         modified = true;
                         SetStrings();
                         break;
@@ -521,7 +521,7 @@ namespace Lucene.Net.Analysis.Fr
                 {
                     if (source.EndsWith(search[i], StringComparison.Ordinal))
                     {
-                        sb.Remove(sb.Length - search[i].Length, sb.Length - 
sb.Length - search[i].Length).Insert(sb.Length - search[i].Length, replace);
+                        sb.Remove(sb.Length - search[i].Length, sb.Length - 
(sb.Length - search[i].Length)).Insert(sb.Length - search[i].Length, replace);
                         modified = true;
                         found = true;
                         SetStrings();