[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [patch] Small fix for java.beans

From: Dalibor Topic
Subject: Re: [patch] Small fix for java.beans
Date: Fri, 24 Oct 2003 16:13:07 +0200
User-agent: Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.3) Gecko/20030312

Oops, and of course I forgot the ChangeLog entry:

2003-10-24  Dalibor Topic <address@hidden>

        * libraries/javalib/gnu/java/beans/
        (addMethod) Add public static methods.

now, flame away ;)

dalibor topic

Dalibor Topic wrote:
Hi all,

while merging in GNU Classpath's java.beans into kaffe, there was a small issue that caused a regression test in kaffe to fail: the Introspector wouldn't add public static methods to the BeanInfo of a bean, as the spec demands. The attached patch solves this problem.

dalibor topic


Index: libraries/javalib/gnu/java/beans/
RCS file: 
retrieving revision 1.2
diff -u -r1.2
--- libraries/javalib/gnu/java/beans/        22 Oct 
2003 10:34:45 -0000      1.2
+++ libraries/javalib/gnu/java/beans/        24 Oct 
2003 13:39:29 -0000
@@ -67,7 +67,7 @@
/* Paving the way for automatic Introspection */
        public void addMethod(Method method) {
-               if(Modifier.isPublic(method.getModifiers()) && 
!Modifier.isStatic(method.getModifiers())) {
+               if(Modifier.isPublic(method.getModifiers())) {
                        String name = 
                        Class retType = method.getReturnType();
                        Class[] params = method.getParameterTypes();


Classpath mailing list

reply via email to

[Prev in Thread] Current Thread [Next in Thread]