nikcio opened a new issue, #675:
URL: https://github.com/apache/lucenenet/issues/675

   Mentioned in #648 
   
   Issues found: 
https://sonarcloud.io/project/issues?resolved=false&rules=external_roslyn%3ACA1846&id=nikcio_lucenenet
   
   We can conditionally reference the package 
[System.Memory](https://www.nuget.org/packages/System.Memory) in `net462` and 
`netstandard2.0` to fix this. Other targets don't need to reference the 
package. No comment needed. This is more efficient and is understood to be a 
better translation from Java to .NET. Do note the overload of `Substring(int, 
int)` differs between Java (second parameter is `end`) and .NET (second 
parameter is `length`. To correct use `length = end - start`. We should have 
done this already.


-- 
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]

Reply via email to