[Mono-bugs] [Bug 79120][Min] New - Warning in Binder.ReorderParameters

[bugzilla-daemon at bugzilla.ximian.com]

Please do not reply to this email- if you want to comment on the bug, go to the
URL shown below and enter your comments there.

Changed by peter.dettman at iinet.net.au.

http://bugzilla.ximian.com/show_bug.cgi?id=79120

--- shadow/79120	2006-08-20 07:28:30.000000000 -0400
+++ shadow/79120.tmp.1748	2006-08-20 07:28:30.000000000 -0400
@@ -0,0 +1,43 @@
+Bug#: 79120
+Product: Mono: Class Libraries
+Version: 1.1
+OS: 
+OS Details: 
+Status: NEW   
+Resolution: 
+Severity: 
+Priority: Minor
+Component: CORLIB
+AssignedTo: mono-bugs at ximian.com                            
+ReportedBy: peter.dettman at iinet.net.au               
+QAContact: mono-bugs at ximian.com
+TargetMilestone: ---
+URL: 
+Cc: 
+Summary: Warning in Binder.ReorderParameters
+
+Description of Problem:
+
+When compiling mcs I noticed the following warning:
+
+System.Reflection/Binder.cs(153,41): warning CS0162: Unreachable code detected
+
+This is an issue with Binder.ReorderParameters.
+
+From a quick look at the code, it seems the 'break' statement should be
+inside the 'if' block, i.e.
+
+
+Index: Binder.cs
+===================================================================
+--- Binder.cs   (revision 64075)
++++ Binder.cs   (working copy)
+@@ -155,8 +155,8 @@
+                            object o = args [n];
+                            args [n] = args [p];
+                            args [p] = o;
++                           break;
+                        }
+-                       break;
+                    }
+            }