On Wed, 7 Jun 2023 05:05:38 GMT, Prasanta Sadhukhan <psadhuk...@openjdk.org> wrote:
>> Alexander Zvegintsev has updated the pull request incrementally with two >> additional commits since the last revision: >> >> - move screencast-tokens.properties to a new location >> - fix failure of >> javax/swing/reliability/HangDuringStaticInitialization.java in headless >> environment > > src/java.desktop/unix/classes/sun/awt/screencast/TokenStorage.java line 145: > >> 143: path, >> 144: Set.of(PosixFilePermission.OWNER_READ, >> 145: PosixFilePermission.OWNER_WRITE) > > alignment issue..Posix..OWNER_WRITE should be placed below Posix..OENER_READ > > Also, why we aren't adding OWNER_EXECUTE here too like done above? Because execute is required on directories but not files unless they are programs. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/13803#discussion_r1220882132