[prev in list] [next in list] [prev in thread] [next in thread] 

List:       mono-patches
Subject:    [Mono-patches] r62154 - trunk/mono/mono/metadata
From:       "Raja R Harinath (rharinath () novell ! com)"
Date:       2006-06-30 10:17:48
Message-ID: 20060630101748.4BDC09472C () mono-cvs ! ximian ! com
[Download RAW message or body]

Author: raja
Date: 2006-06-30 06:17:47 -0400 (Fri, 30 Jun 2006)
New Revision: 62154

Modified:
   trunk/mono/mono/metadata/ChangeLog
   trunk/mono/mono/metadata/icall.c
Log:
* icall.c (ves_icall_MonoMethod_get_base_definition): Simplify
slightly: remove a shadow local variable.

[ I feel compelled to commit this after obsessing over the ugly
  three contiguous '}'.  Yeah, I'm ashamed :-) ]


Modified: trunk/mono/mono/metadata/ChangeLog
===================================================================
--- trunk/mono/mono/metadata/ChangeLog	2006-06-30 10:10:49 UTC (rev 62153)
+++ trunk/mono/mono/metadata/ChangeLog	2006-06-30 10:17:47 UTC (rev 62154)
@@ -1,3 +1,8 @@
+2006-06-30  Raja R Harinath  <rharinath@novell.com>
+
+	* icall.c (ves_icall_MonoMethod_get_base_definition): Simplify
+	slightly: remove a shadow local variable.
+
 2006-06-29  Raja R Harinath  <rharinath@novell.com>
 
 	* icall.c (ves_icall_MonoMethod_get_base_definition): Return the

Modified: trunk/mono/mono/metadata/icall.c
===================================================================
--- trunk/mono/mono/metadata/icall.c	2006-06-30 10:10:49 UTC (rev 62153)
+++ trunk/mono/mono/metadata/icall.c	2006-06-30 10:17:47 UTC (rev 62154)
@@ -6079,15 +6079,11 @@
 
 	result = klass->vtable [method->slot];
 	if (result == NULL) {
-		MonoMethod* m;
-		gpointer iter = NULL;
 		/* It is an abstract method */
-		while ((m = mono_class_get_methods (klass, &iter))) {
-			if (m->slot == method->slot) {
-				result = m;
+		gpointer iter = NULL;
+		while ((result = mono_class_get_methods (klass, &iter)))
+			if (result->slot == method->slot)
 				break;
-			}
-		}
 	}
 
 	if (result == NULL)

_______________________________________________
Mono-patches maillist  -  Mono-patches@lists.ximian.com
http://lists.ximian.com/mailman/listinfo/mono-patches
[prev in list] [next in list] [prev in thread] [next in thread]