Re: [PR] XBEAN-355 - XbeanAsmParameterNameLoader incorrectly maps parameter names when methods contain long or double paramet ers [geronimo-xbean]

rzo1 (via GitHub) <[email protected]> Thu, 05 Feb 2026 12:11:50 -0000
Newsgroups gmane.comp.java.geronimo.devel
Message-ID <PR_kwDOAAOrUs7Bqz_D-ac55487b-8615-4b67-8b2a-16a5b7546968@gitbox.apache.org>
rzo1 commented on code in PR #52:
URL: https://github.com/apache/geronimo-xbean/pull/52#discussion_r2768840370


##########
xbean-reflect/src/test/java/org/apache/xbean/recipe/ParameterNameLoaderTest.java:
##########
@@ -107,6 +107,14 @@ public void testEmptyMethod() throws Exception {
         assertParameterNames(Collections.<String>emptyList(), method);
     }
 
+    public void testXBEAN355() throws  Exception{

Review Comment:
   Yes. Added a test for non static inner class (with a fix for that case) + non static separate class.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe-nmThH5wfGryY1pf4d8vP8EB+6BGkLq7r@public.gmane.org

For queries about this service, please contact Infrastructure at:
users-pF6z+fRhj9EyzMRdD/[email protected]