On Thu, 9 Jan 2025 13:06:55 GMT, Jaikiran Pai <j...@openjdk.org> wrote:
> Can I please get a review of this doc-only change which clarifies the purpose > of the HTTP server that's provided in the JDK's `jdk.httpserver` module? This > should help avoid any confusion on the purpose of the server. src/jdk.httpserver/share/classes/module-info.java line 30: > 28: * for running a minimal HTTP server. > 29: * > 30: * <p>The API, SPI and the server implementation in this module is > intended 1. Add a comma after SPI. ( I had to look up that acronym). 2. Change "is intended" to "are intended." src/jdk.httpserver/share/classes/module-info.java line 32: > 30: * <p>The API, SPI and the server implementation in this module is > intended > 31: * for simple usages like local testing, development, and debugging. > 32: * Accordingly, the design and implementation of the server is explicitly Change "is explicitly" to "are explicitly"? src/jdk.httpserver/share/classes/module-info.java line 33: > 31: * for simple usages like local testing, development, and debugging. > 32: * Accordingly, the design and implementation of the server is explicitly > 33: * minimal so as to avoid confusion with a full-featured and high Replace "and" with a comma? ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/23004#discussion_r1909030875 PR Review Comment: https://git.openjdk.org/jdk/pull/23004#discussion_r1909032183 PR Review Comment: https://git.openjdk.org/jdk/pull/23004#discussion_r1909032901
