paulirwin commented on code in PR #1056:
URL: https://github.com/apache/lucenenet/pull/1056#discussion_r1884101033
##########
src/Lucene.Net.Highlighter/VectorHighlight/FieldPhraseList.cs:
##########
@@ -470,6 +470,31 @@ public override bool Equals(object obj)
return true;
}
+ #region Operator overrides
+ #nullable enable
Review Comment:
Should I do that for the `#region` as well, then? It seems like it would
look weird if the region is indented and the nullable directive is not. We
haven't been consistent, and I'd say I prefer the look of both indented.
--
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-unsubscr...@lucenenet.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
