You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@lucenenet.apache.org by GitBox <gi...@apache.org> on 2022/10/15 19:45:32 UTC

[GitHub] [lucenenet] nikcio opened a new issue, #680: Replace the control character at position 2 by its escape sequence '\u3000'.

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

   Mentioned in #648 
   
   Issues found: https://sonarcloud.io/project/issues?resolved=false&rules=csharpsquid%3AS2479&id=nikcio_lucenenet
   
   This should be marked with a `// LUCENENET specific - changed from invisible whitespace character to '\u3000' for readability`.


-- 
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-unsubscribe@lucenenet.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [lucenenet] NightOwl888 commented on issue #680: Replace the control character at position 2 by its escape sequence '\u3000'.

Posted by GitBox <gi...@apache.org>.
NightOwl888 commented on issue #680:
URL: https://github.com/apache/lucenenet/issues/680#issuecomment-1279934161

   There appears to be a bug in the SonarCloud analyzer report. The `'\u3000'` character is at position 1, not position 2.
   
   ```
   ?(int)"  \t\r\n"[0]
   32
   ?(int)"  \t\r\n"[1]
   12288
   ?(int)"  \t\r\n"[2]
   9
   ?(int)"  \t\r\n"[3]
   13
   ?(int)"  \t\r\n"[4]
   10
   ?(int)'\u3000'
   12288
   ```


-- 
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-unsubscribe@lucenenet.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [lucenenet] NightOwl888 closed issue #680: Replace the control character at position 2 by its escape sequence '\u3000'.

Posted by GitBox <gi...@apache.org>.
NightOwl888 closed issue #680: Replace the control character at position 2 by its escape sequence '\u3000'.
URL: https://github.com/apache/lucenenet/issues/680


-- 
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-unsubscribe@lucenenet.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [lucenenet] nikcio commented on issue #680: Replace the control character at position 2 by its escape sequence '\u3000'.

Posted by GitBox <gi...@apache.org>.
nikcio commented on issue #680:
URL: https://github.com/apache/lucenenet/issues/680#issuecomment-1279934981

   @NightOwl888 I think that it starts counting at 1 instead of 0 and that's why it says 2.


-- 
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-unsubscribe@lucenenet.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org