RFR: 8342281: Deprecate for removal javax.sound.sampled.AudioPermission
Harshitha Onkar
honkar at openjdk.org
Wed Nov 20 22:55:15 UTC 2024
On Wed, 20 Nov 2024 22:21:57 GMT, Phil Race <prr at openjdk.org> wrote:
> Deprecate AudioPermission for removal.
>
> I found a test that uses it and updated it.
>
> The CSR needs to be reviewed too : https://bugs.openjdk.org/browse/JDK-8344658
Changes LGTM except for the SecurityException catch block removal in the test.
test/jdk/javax/sound/sampled/Lines/GetLine.java line 49:
> 47: }
> 48: }
> 49:
Minor:
These are on unchanged lines.
Since AudioSystem.getLine(infos[0]) no longer throws security exception, the SecurityException catch block can be removed.
try {
l = AudioSystem.getLine(infos[0]);
} catch(SecurityException lue) {
log.println("SecurityException");
return STATUS_PASSED;
}
-------------
PR Review: https://git.openjdk.org/jdk/pull/22284#pullrequestreview-2449855057
PR Review Comment: https://git.openjdk.org/jdk/pull/22284#discussion_r1851084514
More information about the client-libs-dev
mailing list