Fix Off-by-one in index mapping when consecutive Chinese chars

VictoriousRaptor · VictoriousRaptor · commit 64a3aa583f72 · 2025-06-19T19:29:50.000+08:00
Found by code rabbit
diff --git a/Flow.Launcher.Infrastructure/PinyinAlphabet.cs b/Flow.Launcher.Infrastructure/PinyinAlphabet.cs
@@ -107,8 +107,8 @@ public bool ShouldTranslate(string stringToTranslate)
                     string translated = _settings.UseDoublePinyin ? ToDoublePin(resultList[i]) : resultList[i];
                     if (previousIsChinese)
                     {
-                        map.AddNewIndex(i, resultBuilder.Length, translated.Length + 1);
                         resultBuilder.Append(' ');
+                        map.AddNewIndex(i, resultBuilder.Length, translated.Length + 1);
                         resultBuilder.Append(translated);
                     }
                     else

Original file line number	Diff line number	Diff line change
`@@ -107,8 +107,8 @@ public bool ShouldTranslate(string stringToTranslate)`
`107`	`107`	`string translated = _settings.UseDoublePinyin ? ToDoublePin(resultList[i]) : resultList[i];`
`108`	`108`	`if (previousIsChinese)`
`109`	`109`	`{`
`110`		`- map.AddNewIndex(i, resultBuilder.Length, translated.Length + 1);`
`111`	`110`	`resultBuilder.Append(' ');`
	`111`	`+ map.AddNewIndex(i, resultBuilder.Length, translated.Length + 1);`
`112`	`112`	`resultBuilder.Append(translated);`
`113`	`113`	`}`
`114`	`114`	`else`