Re: RFR: 8344185: Remove calls to SecurityManager in sun.net.ftp

2024-11-15 Thread Daniel Fuchs
On Thu, 14 Nov 2024 12:57:25 GMT, Eirik Bjørsnøs wrote: > Please review this PR which removes uses of `AccessController::doPrivileged` > in `sun/net/ftp`. > > Changes are mostly straightforward, one exception is the removal of the > somewhat exotic unwrapping of temporary arrays in the static

Re: RFR: 8344185: Remove calls to SecurityManager in sun.net.ftp

2024-11-14 Thread Eirik Bjørsnøs
On Thu, 14 Nov 2024 12:57:25 GMT, Eirik Bjørsnøs wrote: > Please review this PR which removes uses of `AccessController::doPrivileged` > in `sun/net/ftp`. > > Changes are mostly straightforward, one exception is the removal of the > somewhat exotic unwrapping of temporary arrays in the static

Re: RFR: 8344185: Remove calls to SecurityManager in sun.net.ftp

2024-11-14 Thread Michael McMahon
On Thu, 14 Nov 2024 12:57:25 GMT, Eirik Bjørsnøs wrote: > Please review this PR which removes uses of `AccessController::doPrivileged` > in `sun/net/ftp`. > > Changes are mostly straightforward, one exception is the removal of the > somewhat exotic unwrapping of temporary arrays in the static

Re: RFR: 8344185: Remove calls to SecurityManager in sun.net.ftp

2024-11-14 Thread Daniel Fuchs
On Thu, 14 Nov 2024 12:57:25 GMT, Eirik Bjørsnøs wrote: > Please review this PR which removes uses of `AccessController::doPrivileged` > in `sun/net/ftp`. > > Changes are mostly straightforward, one exception is the removal of the > somewhat exotic unwrapping of temporary arrays in the static

Re: RFR: 8344185: Remove calls to SecurityManager in sun.net.ftp

2024-11-14 Thread Alan Bateman
On Thu, 14 Nov 2024 12:57:25 GMT, Eirik Bjørsnøs wrote: > Please review this PR which removes uses of `AccessController::doPrivileged` > in `sun/net/ftp`. > > Changes are mostly straightforward, one exception is the removal of the > somewhat exotic unwrapping of temporary arrays in the static

RFR: 8344185: Remove calls to SecurityManager in sun.net.ftp

2024-11-14 Thread Eirik Bjørsnøs
Please review this PR which removes uses of `AccessController::doPrivileged` in `sun/net/ftp`. Changes are mostly straightforward, one exception is the removal of the somewhat exotic unwrapping of temporary arrays in the static initializer of `impl/FtpClient`. Testing: `test/jdk/sun/net/ftp` r