Re: RFR: 8270380: Change the default value of the java.security.manager system property to disallow

2021-08-20 Thread Lance Andersen
On Fri, 20 Aug 2021 22:44:34 GMT, Weijun Wang wrote: > This change modifies the default value of the `java.security.manager` system > property from "allow" to "disallow". This means unless it's explicitly set to > "allow", any call to `System.setSecurityManager()` would throw an UOE. > > The `

RFR: 8270380: Change the default value of the java.security.manager system property to disallow

2021-08-20 Thread Weijun Wang
This change modifies the default value of the `java.security.manager` system property from "allow" to "disallow". This means unless it's explicitly set to "allow", any call to `System.setSecurityManager()` would throw an UOE. The `AllowSecurityManager.java` and `SecurityManagerWarnings.java` tes

Re: RFR: 8271302: Regex Test Refresh [v4]

2021-08-20 Thread Ian Graves
> 8271302: Regex Test Refresh Ian Graves has updated the pull request incrementally with one additional commit since the last revision: Additional cleanup - Changes: - all: https://git.openjdk.java.net/jdk/pull/5092/files - new: https://git.openjdk.java.net/jdk/pull/5092/file

Re: RFR: 8271302: Regex Test Refresh [v3]

2021-08-20 Thread Pavel Rappo
On Fri, 20 Aug 2021 16:49:15 GMT, Pavel Rappo wrote: >> Ian Graves has updated the pull request incrementally with one additional >> commit since the last revision: >> >> some quick fixes > > test/jdk/java/util/regex/RegExTest.java line 4270: > >> 4268: String s = (String)pm[1];

Re: RFR: 8271302: Regex Test Refresh [v3]

2021-08-20 Thread Pavel Rappo
On Fri, 20 Aug 2021 16:27:53 GMT, Ian Graves wrote: >> 8271302: Regex Test Refresh > > Ian Graves has updated the pull request incrementally with one additional > commit since the last revision: > > some quick fixes test/jdk/java/util/regex/RegExTest.java line 4270: > 4268: Stri

Re: RFR: 8271302: Regex Test Refresh [v3]

2021-08-20 Thread Ian Graves
> 8271302: Regex Test Refresh Ian Graves has updated the pull request incrementally with one additional commit since the last revision: some quick fixes - Changes: - all: https://git.openjdk.java.net/jdk/pull/5092/files - new: https://git.openjdk.java.net/jdk/pull/5092/files/

Re: RFR: 8271302: Regex Test Refresh [v2]

2021-08-20 Thread Ian Graves
On Fri, 20 Aug 2021 13:32:24 GMT, Pavel Rappo wrote: >> Ian Graves has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Couple of fixes > > test/jdk/java/util/regex/NegativeArraySize.java line 29: > >> 27: * @summary Pattern.compile() can t

Re: RFR: 8271302: Regex Test Refresh [v2]

2021-08-20 Thread Pavel Rappo
On Fri, 20 Aug 2021 13:46:39 GMT, Pavel Rappo wrote: >> Ian Graves has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Couple of fixes > > test/jdk/java/util/regex/NegativeArraySize.java line 40: > >> 38: @Test >> 39: public static

Re: what does the spec say about file paths that are too long?

2021-08-20 Thread Alan Snyder
Where did I say that I was seeing silent truncation? My question is about the specification. I want to know if I should write code that defends against silent truncation. I should be writing code based on the specification, not what I observe in a test program on a particular platform. I’m ple

Re: RFR: 8271302: Regex Test Refresh [v2]

2021-08-20 Thread Pavel Rappo
On Wed, 18 Aug 2021 18:35:53 GMT, Ian Graves wrote: >> 8271302: Regex Test Refresh > > Ian Graves has updated the pull request incrementally with one additional > commit since the last revision: > > Couple of fixes test/jdk/java/util/regex/NegativeArraySize.java line 2: > 1: /* > 2: * Copy

Re: [jdk17] RFR: 8270916: Update java.lang.annotation.Target for changes in JLS 9.6.4.1

2021-08-20 Thread Guoxiong Li
On Thu, 22 Jul 2021 19:38:07 GMT, Joe Darcy wrote: >> Given changes in JLS 9.6.4.1, JDK-8261610 in Java SE 17, the statements >> about annotation applicability made in java.lang.annotation.Target need to >> be updated to match. >> >> Please also review the corresponding CSR: >> https://bugs.o

Re: Process for adding default method to javax.naming.Context?

2021-08-20 Thread Alan Bateman
On 19/08/2021 15:32, Laird Nelson wrote: : To recap in brief: being able to do: Frob frob = someNamingContext.lookup("frob", Frob.class); would be very convenient, e.g. via something like: // Proposed new default method in javax.naming.Context default T lookup(Name name, Class type)

Re: what does the spec say about file paths that are too long?

2021-08-20 Thread Alan Bateman
On 19/08/2021 22:52, Alan Snyder wrote: I’ve been looking for some specification of what happens when a file request is made using a path that is too long (whatever that means to the underlying system). I have not found one. Is there one? My opinion is that silent truncation should not happen.