nikcio opened a new issue, #719: URL: https://github.com/apache/lucenenet/issues/719
The following code has triggered an SonarCloud issue: https://github.com/apache/lucenenet/blob/b6cf28b78c85f33e66d89abe399c4523dd41dc80/src/Lucene.Net.Analysis.ICU/Analysis/Icu/ICUTransformFilter.cs#L135-L140 The `Replace` statement has a 2nd paramenter that always defaults to 0. It seems to be a bug. Original code: https://github.com/apache/lucene/blob/8fdf89690404c0e65784b2c5477552b9dec58591/lucene/analysis/icu/src/java/org/apache/lucene/analysis/icu/ICUTransformFilter.java#L138-L142 Traced the issue back to: https://github.com/apache/lucenenet/commit/bebbae94145916520972ce25b3ebc2778db7c699#diff-ff3f44867a5688ccfea635f8d3c39fca938e698e60df8b6016a329a0e32fb40c Before this commit it seems that it was `Replace(dest, dest....` instead of `Replace(dest, dest - dest...`. So it might be a mistake Sonar issue: https://sonarcloud.io/project/issues?issues=AYPAuQyghbfJOGLOocHG&open=AYPAuQyghbfJOGLOocHG&id=nikcio_lucenenet -- 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.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
