Re: RFR: 8344365: SecurityManager cleanups in java.sql and java.sql.rowset modules [v2]

2024-11-18 Thread Eirik Bjørsnøs
On Mon, 18 Nov 2024 19:30:08 GMT, Roger Riggs wrote: >> Eirik Bjørsnøs has updated the pull request incrementally with two >> additional commits since the last revision: >> >> - Revert caller-sensitive inlining of ReflectUtil.forName >> - SerialJavaObject::getFields is no longer @CallerSensit

Re: RFR: 8344365: SecurityManager cleanups in java.sql and java.sql.rowset modules [v2]

2024-11-18 Thread Roger Riggs
On Mon, 18 Nov 2024 08:37:26 GMT, Eirik Bjørsnøs wrote: >> Please review this PR which cleans up SecurityManager-related code in >> `java.sql` and `java.sql.rowset` modules post JEP-486 >> >> There are quite a few changes to review, but all relatively straightforward: >> >> `DriverManager` >>

Re: RFR: 8344365: SecurityManager cleanups in java.sql and java.sql.rowset modules [v2]

2024-11-18 Thread Lance Andersen
On Mon, 18 Nov 2024 08:37:26 GMT, Eirik Bjørsnøs wrote: >> Please review this PR which cleans up SecurityManager-related code in >> `java.sql` and `java.sql.rowset` modules post JEP-486 >> >> There are quite a few changes to review, but all relatively straightforward: >> >> `DriverManager` >>

Re: RFR: 8344365: SecurityManager cleanups in java.sql and java.sql.rowset modules [v2]

2024-11-18 Thread Eirik Bjørsnøs
> Please review this PR which cleans up SecurityManager-related code in > `java.sql` and `java.sql.rowset` modules post JEP-486 > > There are quite a few changes to review, but all relatively straightforward: > > `DriverManager` > * Remove `SecurityManager::checkPermission` calls in the `setLogW