Re: RFR: 8353917: jnativescan: Simplify ClassResolver [v3]

2025-04-11 Thread Jorn Vernee
On Thu, 10 Apr 2025 17:07:21 GMT, Jorn Vernee wrote: >> `jnativescan` uses the `ClassResolver` class to find both system classes, as >> well as application classes. In principle, a class resolver supports both >> iterating over all classes from that particular source, as well as looking >> up

Re: RFR: 8353917: jnativescan: Simplify ClassResolver [v3]

2025-04-11 Thread Maurizio Cimadamore
On Thu, 10 Apr 2025 17:07:21 GMT, Jorn Vernee wrote: >> `jnativescan` uses the `ClassResolver` class to find both system classes, as >> well as application classes. In principle, a class resolver supports both >> iterating over all classes from that particular source, as well as looking >> up

Re: RFR: 8353917: jnativescan: Simplify ClassResolver [v2]

2025-04-10 Thread Maurizio Cimadamore
On Wed, 9 Apr 2025 16:00:52 GMT, Jorn Vernee wrote: >> `jnativescan` uses the `ClassResolver` class to find both system classes, as >> well as application classes. In principle, a class resolver supports both >> iterating over all classes from that particular source, as well as looking >> up c

Re: RFR: 8353917: jnativescan: Simplify ClassResolver [v2]

2025-04-10 Thread Jorn Vernee
On Thu, 10 Apr 2025 14:48:59 GMT, Maurizio Cimadamore wrote: >> Jorn Vernee has updated the pull request with a new target base due to a >> merge or a rebase. The pull request now contains four commits: >> >> - Merge branch 'master' into jnativescan_Refactor >> - bump copyright years >> - m

Re: RFR: 8353917: jnativescan: Simplify ClassResolver [v3]

2025-04-10 Thread Jorn Vernee
> `jnativescan` uses the `ClassResolver` class to find both system classes, as > well as application classes. In principle, a class resolver supports both > iterating over all classes from that particular source, as well as looking up > classes by name. However, the `ClassResolver` for system cl

Re: RFR: 8353917: jnativescan: Simplify ClassResolver [v2]

2025-04-10 Thread Maurizio Cimadamore
On Wed, 9 Apr 2025 16:00:52 GMT, Jorn Vernee wrote: >> `jnativescan` uses the `ClassResolver` class to find both system classes, as >> well as application classes. In principle, a class resolver supports both >> iterating over all classes from that particular source, as well as looking >> up c

Re: RFR: 8353917: jnativescan: Simplify ClassResolver [v2]

2025-04-10 Thread Maurizio Cimadamore
On Thu, 10 Apr 2025 14:43:20 GMT, Maurizio Cimadamore wrote: > Very nice simplification! This seems to take advantage of the fact that, for the restricted method analysis, we only need to look at method reference entries whose owner is a system class -- because we know restricted methods are

Re: RFR: 8353917: jnativescan: Simplify ClassResolver [v2]

2025-04-09 Thread Jorn Vernee
> `jnativescan` uses the `ClassResolver` class to find both system classes, as > well as application classes. In principle, a class resolver supports both > iterating over all classes from that particular source, as well as looking up > classes by name. However, the `ClassResolver` for system cl

Re: RFR: 8353917: jnativescan: Simplify ClassResolver

2025-04-08 Thread Jorn Vernee
On Mon, 7 Apr 2025 16:36:07 GMT, Jorn Vernee wrote: > `jnativescan` uses the `ClassResolver` class to find both system classes, as > well as application classes. In principle, a class resolver supports both > iterating over all classes from that particular source, as well as looking up > class

RFR: 8353917: jnativescan: Simplify ClassResolver

2025-04-08 Thread Jorn Vernee
`jnativescan` uses the `ClassResolver` class to find both system classes, as well as application classes. In principle, a class resolver supports both iterating over all classes from that particular source, as well as looking up classes by name. However, the `ClassResolver` for system classes do