On Mon, 25 Aug 2025 14:51:43 GMT, Chen Liang <li...@openjdk.org> wrote:
>> A previous cleanup #14642 accidentally omitted the fact that an >> `Array.newInstance` call in `BytecodeDescriptor::parseSig` is intended to >> propagate `IllegalArgumentException` to >> `MethodType::fromMethodDescriptorString`. This bug is discovered in a recent >> cleanup for `BytecodeDescriptor` in #24978. >> >> Instead of restoring the original `newInstance` call, this patch catches the >> `UnsupportedOperationException` thrown by `Class::arrayType` to because the >> cause IAE thrown by `newInstance` has no message at all. The existing >> reporting system in `BytecodeDescriptor::parseMethod` includes the whole >> malformed descriptor string in the error message, which can be triggered by >> returning `null` in `parseSig`. > > Chen Liang has updated the pull request incrementally with one additional > commit since the last revision: > > A case for malformed return type Marked as reviewed by jvernee (Reviewer). ------------- PR Review: https://git.openjdk.org/jdk/pull/26909#pullrequestreview-3151833079
