msokolov commented on code in PR #14019: URL: https://github.com/apache/lucene/pull/14019#discussion_r1864957737
########## lucene/core/src/java/org/apache/lucene/index/DocumentsWriterPerThreadPool.java: ########## @@ -184,7 +184,7 @@ List<DocumentsWriterPerThread> filterAndLock(Predicate<DocumentsWriterPerThread> /** * Removes the given DWPT from the pool unless it's already been removed before. * - * @return <code>true</code> iff the given DWPT has been removed. Otherwise <code>false</code> + * @return <code>true</code>, iff the given DWPT has been removed. Otherwise <code>false</code> Review Comment: Since this is all about syntax, grammar, etc, we might as well italicize _iff_ since that is how it is generally done, as in the example you cited. -- 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