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
