SF.net SVN: jikesrvm:[15692] rvmroot/trunk/libraryInterface/Common/src/java /lang/Class.java

View: New views
2 Messages — Rating Filter:   Alert me  

SF.net SVN: jikesrvm:[15692] rvmroot/trunk/libraryInterface/Common/src/java /lang/Class.java

by dframpton-oss :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Revision: 15692
          http://jikesrvm.svn.sourceforge.net/jikesrvm/?rev=15692&view=rev
Author:   dframpton-oss
Date:     2009-05-20 04:30:11 +0000 (Wed, 20 May 2009)

Log Message:
-----------
Do not search superclasses for a method unless no method is found.

Modified Paths:
--------------
    rvmroot/trunk/libraryInterface/Common/src/java/lang/Class.java

Modified: rvmroot/trunk/libraryInterface/Common/src/java/lang/Class.java
===================================================================
--- rvmroot/trunk/libraryInterface/Common/src/java/lang/Class.java 2009-05-20 01:46:10 UTC (rev 15691)
+++ rvmroot/trunk/libraryInterface/Common/src/java/lang/Class.java 2009-05-20 04:30:11 UTC (rev 15692)
@@ -877,7 +877,7 @@
   @Pure
   private RVMMethod getMethodInternal1(Atom aName, Class<?>... parameterTypes) {
     RVMMethod answer = null;
-    for (RVMClass current = type.asClass(); current != null; current = current.getSuperClass()) {
+    for (RVMClass current = type.asClass(); current != null && answer == null; current = current.getSuperClass()) {
       RVMMethod[] methods = current.getDeclaredMethods();
       for (RVMMethod meth : methods) {
         if (meth.getName() == aName && meth.isPublic() &&


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.

------------------------------------------------------------------------------
Crystal Reports - New Free Runtime and 30 Day Trial
Check out the new simplified licensing option that enables
unlimited royalty-free distribution of the report engine
for externally facing server and web deployment.
http://p.sf.net/sfu/businessobjects
_______________________________________________
Jikesrvm-commits mailing list
Jikesrvm-commits@...
https://lists.sourceforge.net/lists/listinfo/jikesrvm-commits

Re: SF.net SVN: jikesrvm:[15692] rvmroot/trunk/libraryInterface/Common/src/java /lang/Class.java

by Ian Rogers (nabble) :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Hi,

when taking code from MRP to the RVM (it's great the RVM can catch up
with us) can you make sure that the commit message attribute the code
to MRP. This patch is MRP-2.

Thanks,
Ian

2009/5/19  <dframpton-oss@...>:

> Revision: 15692
>          http://jikesrvm.svn.sourceforge.net/jikesrvm/?rev=15692&view=rev
> Author:   dframpton-oss
> Date:     2009-05-20 04:30:11 +0000 (Wed, 20 May 2009)
>
> Log Message:
> -----------
> Do not search superclasses for a method unless no method is found.
>
> Modified Paths:
> --------------
>    rvmroot/trunk/libraryInterface/Common/src/java/lang/Class.java
>
> Modified: rvmroot/trunk/libraryInterface/Common/src/java/lang/Class.java
> ===================================================================
> --- rvmroot/trunk/libraryInterface/Common/src/java/lang/Class.java      2009-05-20 01:46:10 UTC (rev 15691)
> +++ rvmroot/trunk/libraryInterface/Common/src/java/lang/Class.java      2009-05-20 04:30:11 UTC (rev 15692)
> @@ -877,7 +877,7 @@
>   @Pure
>   private RVMMethod getMethodInternal1(Atom aName, Class<?>... parameterTypes) {
>     RVMMethod answer = null;
> -    for (RVMClass current = type.asClass(); current != null; current = current.getSuperClass()) {
> +    for (RVMClass current = type.asClass(); current != null && answer == null; current = current.getSuperClass()) {
>       RVMMethod[] methods = current.getDeclaredMethods();
>       for (RVMMethod meth : methods) {
>         if (meth.getName() == aName && meth.isPublic() &&
>
>
> This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.
>
> ------------------------------------------------------------------------------
> Crystal Reports - New Free Runtime and 30 Day Trial
> Check out the new simplified licensing option that enables
> unlimited royalty-free distribution of the report engine
> for externally facing server and web deployment.
> http://p.sf.net/sfu/businessobjects
> _______________________________________________
> Jikesrvm-commits mailing list
> Jikesrvm-commits@...
> https://lists.sourceforge.net/lists/listinfo/jikesrvm-commits
>

------------------------------------------------------------------------------
Crystal Reports - New Free Runtime and 30 Day Trial
Check out the new simplified licensing option that enables
unlimited royalty-free distribution of the report engine
for externally facing server and web deployment.
http://p.sf.net/sfu/businessobjects
_______________________________________________
Jikesrvm-commits mailing list
Jikesrvm-commits@...
https://lists.sourceforge.net/lists/listinfo/jikesrvm-commits