On Wed, 17 Jan 2024 18:19:18 GMT, Naoto Sato <na...@openjdk.org> wrote:
>> src/java.base/share/classes/java/util/TimeZone.java line 629: >> >>> 627: */ >>> 628: public static String[] getAvailableIDs(int rawOffset) { >>> 629: return ZoneInfo.getAvailableIDs(rawOffset); >> >> BTW can we call `ZoneInfoFile.getZoneIds` here directly? >> `ZoneInfo.getAvailableIDs` bridge seems redudnant. > > `TimeZone` is an abstract class, and `ZoneInfo` is an implementation. So to > me, it is clearer to call the overriden method in the implementation even > though the reason you mentioned. >From OOP point of view - it's strange to call subclass method from a super >class. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/17441#discussion_r1465516080