[Mono-bugs] [Bug 69999][Min] New - BigInteger: ModulusRing.OddPow doing double work?

bugzilla-daemon@bugzilla.ximian.com bugzilla-daemon@bugzilla.ximian.com
Mon, 29 Nov 2004 02:27:51 -0500 (EST)


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 pieter@mentalis.org.

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

--- shadow/69999	2004-11-29 02:27:50.000000000 -0500
+++ shadow/69999.tmp.9788	2004-11-29 02:27:50.000000000 -0500
@@ -0,0 +1,22 @@
+Bug#: 69999
+Product: Mono: Class Libraries
+Version: unspecified
+OS: 
+OS Details: 
+Status: NEW   
+Resolution: 
+Severity: 
+Priority: Minor
+Component: System.Security
+AssignedTo: mono-bugs@ximian.com                            
+ReportedBy: pieter@mentalis.org               
+QAContact: mono-bugs@ximian.com
+TargetMilestone: ---
+URL: 
+Cc: 
+Summary: BigInteger: ModulusRing.OddPow doing double work?
+
+The final Montgomery.Reduce in the ModulusRing.OddPow method seems 
+unnecessary to me. As far as I can see, resultNum will always be reduced 
+before it leaves the loop.
+Is there a reason that there's a reduce after the for loop?