On Mon, 13 Oct 2025 20:34:20 GMT, Matias Saavedra Silva <[email protected]> 
wrote:

>> Trivial change to remove an unused line of code. Verified with tier1-5 tests.
>
> Matias Saavedra Silva has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Removed unused argument

Aside from the JVM_FindClassFromLoader comment, this looks okay to me.

src/hotspot/share/include/jvm.h line 440:

> 438:  */
> 439: JNIEXPORT jclass JNICALL
> 440: JVM_FindClassFromLoader(JNIEnv *env, const char *name, jboolean init,

The function description still lists the caller class, you can remove that.

-------------

Marked as reviewed by alanb (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/27735#pullrequestreview-3334042977
PR Review Comment: https://git.openjdk.org/jdk/pull/27735#discussion_r2428019210

Reply via email to