dweiss commented on code in PR #12973: URL: https://github.com/apache/lucene/pull/12973#discussion_r1444446785
########## lucene/queryparser/src/java/org/apache/lucene/queryparser/flexible/standard/parser/EscapeQuerySyntaxImpl.java: ########## @@ -184,7 +186,7 @@ public CharSequence escape(CharSequence text, Locale locale, Type type) { * Returns a String where the escape char has been removed, or kept only once if there was a * double escape. * - * <p>Supports escaped unicode characters, e. g. translates <code>A</code> to <code>A</code>. + * <p>Supports escaped Unicode characters, e.g. translates <code>A</code> to <code>A</code>. Review Comment: Can you check if the newer ```{@code xxx}``` markup works correctly here? Sorry, I'm busy at the moment. Also, I would not worry about intellij - if javadoc produces valid output, intellij has a bug (feel free to report it!). Escape sequences may be translated very early by javac lexer - I'm sure there is a way to escape them though. -- 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: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org