Re: [patch] Small fix for java.beans

[Dalibor Topic]

Oops, and of course I forgot the ChangeLog entry:

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

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

now, flame away ;)

cheers,
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.
>
> cheers,
> dalibor topic
>
>
> ------------------------------------------------------------------------
>
> Index: libraries/javalib/gnu/java/beans/IntrospectionIncubator.java
> ===================================================================
> RCS file: 
> /cvs/kaffe/kaffe/libraries/javalib/gnu/java/beans/IntrospectionIncubator.java,v
> retrieving revision 1.2
> diff -u -r1.2 IntrospectionIncubator.java
> --- libraries/javalib/gnu/java/beans/IntrospectionIncubator.java        22 Oct 
> 2003 10:34:45 -0000      1.2
> +++ libraries/javalib/gnu/java/beans/IntrospectionIncubator.java        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 = 
> ClassHelper.getTruncatedName(method.getName());
>                         Class retType = method.getReturnType();
>                         Class[] params = method.getParameterTypes();
>
>
> ------------------------------------------------------------------------
>
> _______________________________________________
> Classpath mailing list
> address@hidden
> http://mail.gnu.org/mailman/listinfo/classpath