RFR: Updated codeowners

[Alan Bateman]

On Tue, 20 Feb 2024 17:49:23 GMT, Jesper Wilhelmsson <jwilhelm at openjdk.org> wrote:

> The codeowners list was updated with the latest new and removed directories. Also added links to directories.

Looks fine, a few suggestions included.

src/guide/code-owners.md line 73:

> 71:       * [foreign](https://github.com/openjdk/jdk/tree/master/src/java.base/share/classes/jdk/internal/foreign) - Core Libs
> 72:       * [icu](https://github.com/openjdk/jdk/tree/master/src/java.base/share/classes/jdk/internal/icu) - Core Libs
> 73:       * [io](https://github.com/openjdk/jdk/tree/master/src/java.base/share/classes/jdk/internal/io) - NIO

This should be Core Libs, not NIO.

src/guide/code-owners.md line 75:

> 73:       * [io](https://github.com/openjdk/jdk/tree/master/src/java.base/share/classes/jdk/internal/io) - NIO
> 74:       * [javac](https://github.com/openjdk/jdk/tree/master/src/java.base/share/classes/jdk/internal/javac) - Java Language (javac)
> 75:       * [jimage](https://github.com/openjdk/jdk/tree/master/src/java.base/share/classes/jdk/internal/jimage) - Client Libs

This should Core Libs, not Client Libs.

src/guide/code-owners.md line 76:

> 74:       * [javac](https://github.com/openjdk/jdk/tree/master/src/java.base/share/classes/jdk/internal/javac) - Java Language (javac)
> 75:       * [jimage](https://github.com/openjdk/jdk/tree/master/src/java.base/share/classes/jdk/internal/jimage) - Client Libs
> 76:       * [jmod](https://github.com/openjdk/jdk/tree/master/src/java.base/share/classes/jdk/internal/jmod) - HotSpot

This should be Core Libs.

src/guide/code-owners.md line 94:

> 92:       * [vm](https://github.com/openjdk/jdk/tree/master/src/java.base/share/classes/jdk/internal/vm) - HotSpot
> 93:     * [invoke](https://github.com/openjdk/jdk/tree/master/src/java.base/share/classes/sun/invoke) - Core Libs
> 94:     * [[share](https://github.com/openjdk/jdk/tree/master/java.base/share/classes/java/io), [sun](https://github.com/openjdk/jdk/tree/master/java.base/windows/classes/sun/io), [unix](https://github.com/openjdk/jdk/tree/master/java.base/unix/classes/java/io)]/io - NIO

This should be Core Libs.

src/guide/code-owners.md line 99:

> 97:       * [math](https://github.com/openjdk/jdk/tree/master/src/java.base/share/classes/java/math) - Core Libs
> 98:       * [time](https://github.com/openjdk/jdk/tree/master/src/java.base/share/classes/java/time) - Core Libs
> 99:     * [launcher](https://github.com/openjdk/jdk/tree/master/src/java.base/share/classes/sun/launcher) - Tools

That may be where the historical ownership was but any changes to the launcher tend to be discussed/reviews on core-libs so probably better to change this owner.

src/guide/code-owners.md line 122:

> 120:     * [[aix](https://github.com/openjdk/jdk/tree/master/java.base/aix/native/libjava), [linux](https://github.com/openjdk/jdk/tree/master/java.base/linux/native/libjava), [macosx](https://github.com/openjdk/jdk/tree/master/java.base/macosx/native/libjava), [share](https://github.com/openjdk/jdk/tree/master/java.base/share/native/libjava), [unix](https://github.com/openjdk/jdk/tree/master/java.base/unix/native/libjava), [windows](https://github.com/openjdk/jdk/tree/master/java.base/windows/native/libjava)]/libjava - Core Libs
> 121:     * [[share](https://github.com/openjdk/jdk/tree/master/java.base/share/native/libjimage), [unix](https://github.com/openjdk/jdk/tree/master/java.base/unix/native/libjimage), [windows](https://github.com/openjdk/jdk/tree/master/java.base/windows/native/libjimage)]/libjimage - Core Libs
> 122:     * [[aix](https://github.com/openjdk/jdk/tree/master/java.base/aix/native/libjli), [macosx](https://github.com/openjdk/jdk/tree/master/java.base/macosx/native/libjli), [share](https://github.com/openjdk/jdk/tree/master/java.base/share/native/libjli), [unix](https://github.com/openjdk/jdk/tree/master/java.base/unix/native/libjli), [windows](https://github.com/openjdk/jdk/tree/master/java.base/windows/native/libjli)]/libjli - Tools

"jli" = Java launcher infrastructure so should have same owner as java launcher.

-------------

Marked as reviewed by alanb (Author).

PR Review: https://git.openjdk.org/guide/pull/120#pullrequestreview-1899234305
PR Review Comment: https://git.openjdk.org/guide/pull/120#discussion_r1501363990
PR Review Comment: https://git.openjdk.org/guide/pull/120#discussion_r1501364019
PR Review Comment: https://git.openjdk.org/guide/pull/120#discussion_r1501364089
PR Review Comment: https://git.openjdk.org/guide/pull/120#discussion_r1501364176
PR Review Comment: https://git.openjdk.org/guide/pull/120#discussion_r1501364483
PR Review Comment: https://git.openjdk.org/guide/pull/120#discussion_r1501364625