From 404727f3de51e31014906b2d4a85030614b7c6cb Mon Sep 17 00:00:00 2001
From: Tomasz Sowa <t.sowa@ttmath.org>
Date: Sat, 7 Mar 2009 22:54:42 +0000
Subject: [PATCH] changed: small changes in: UInt:: RclMoveAllWords,
 UInt::RcrMoveAllWords, UInt::SetBitInWord(),         
 UInt::FindLeadingBitInWord, UInt::SetBitInWord fixed:   UInt::Div() didn't
 return a correct result when the divisor was equal 1          there was an
 error in UInt::DivInt() - when the divisor was 1 it returned          zero
 and the carry was set          this error was from the beginning of the
 TTMath library

git-svn-id: svn://ttmath.org/publicrep/ttmath/trunk@103 e52654a7-88a9-db11-a3e9-0013d4bc506e
---
 ttmath/ttmathuint.h | 40 ++++++++++++++++++++--------------------
 1 file changed, 20 insertions(+), 20 deletions(-)

diff --git a/ttmath/ttmathuint.h b/ttmath/ttmathuint.h
index d507564..7642060 100644
--- a/ttmath/ttmathuint.h
+++ b/ttmath/ttmathuint.h
@@ -1207,15 +1207,16 @@ private:
 	*/
 	void RclMoveAllWords(uint & rest_bits, uint & last_c, uint bits, uint c)
 	{
-		rest_bits      = sint(bits % TTMATH_BITS_PER_UINT);
-		sint all_words = sint(bits / TTMATH_BITS_PER_UINT);
+		rest_bits      = bits % TTMATH_BITS_PER_UINT;
+		uint all_words = bits / TTMATH_BITS_PER_UINT;
 		uint mask      = ( c ) ? TTMATH_UINT_MAX_VALUE : 0;
 
 
-		if( all_words >= sint(value_size) )
+		if( all_words >= value_size )
 		{
 			if( all_words == value_size && rest_bits == 0 )
 				last_c = table[0] & 1;
+			// else: last_c is default set to 0
 
 			// clearing
 			for(uint i = 0 ; i<value_size ; ++i)
@@ -1296,15 +1297,16 @@ private:
 	*/
 	void RcrMoveAllWords(uint & rest_bits, uint & last_c, uint bits, uint c)
 	{
-		rest_bits      = sint(bits % TTMATH_BITS_PER_UINT);
-		sint all_words = sint(bits / TTMATH_BITS_PER_UINT);
-		uint mask      = c ? TTMATH_UINT_MAX_VALUE : 0;
+		rest_bits      = bits % TTMATH_BITS_PER_UINT;
+		uint all_words = bits / TTMATH_BITS_PER_UINT;
+		uint mask      = ( c ) ? TTMATH_UINT_MAX_VALUE : 0;
 
 
-		if( all_words >= sint(value_size) )
+		if( all_words >= value_size )
 		{
 			if( all_words == value_size && rest_bits == 0 )
 				last_c = (table[value_size-1] & TTMATH_UINT_HIGHEST_BIT) ? 1 : 0;
+			// else: last_c is default set to 0
 
 			// clearing
 			for(uint i = 0 ; i<value_size ; ++i)
@@ -1436,9 +1438,9 @@ public:
 				push edx
 
 				mov edx,-1
-				bsr eax,x 
+				bsr eax,[x]
 				cmovz eax,edx
-				mov result, eax
+				mov [result], eax
 
 				pop edx
 				pop eax
@@ -1521,10 +1523,10 @@ public:
 			{
 			push ebx
 			push eax
-			mov eax, value
-			mov ebx, bit
+			mov eax, [value]
+			mov ebx, [bit]
 			bts eax, ebx
-			mov value, eax
+			mov [value], eax
 			pop eax
 			pop ebx
 			}
@@ -1534,10 +1536,10 @@ public:
 		#ifdef __GNUC__
 			__asm__  __volatile__(
 
-			"btsl %%ebx,%%eax	\n"
+			"btsl %2,%0		\n"
 
-			: "=a" (value)
-			: "0" (value), "b" (bit)
+			: "=R" (value)
+			: "0" (value), "R" (bit)
 			: "cc" );
 
 		#endif
@@ -1656,7 +1658,7 @@ public:
 	
 		this method never returns a carry
 
-		it is an auxiliary method for version two of the multiplication algorithm
+		it is an auxiliary method for second version of the multiplication algorithm
 	*/
 	static void MulTwoWords(uint a, uint b, uint * result2, uint * result1)
 	{
@@ -1664,7 +1666,7 @@ public:
 		we must use these temporary variables in order to inform the compilator
 		that value pointed with result1 and result2 has changed
 
-		this has no effect in visual studio but it's usefull when
+		this has no effect in visual studio but it's useful when
 		using gcc and options like -Ox
 	*/
 	register uint result1_;
@@ -2044,12 +2046,10 @@ public:
 	{
 		if( divisor == 1 )
 		{
-			SetZero();
-
 			if( remainder )
 				*remainder = 0;
 
-		return 1;
+		return 0;
 		}
 
 		UInt<value_size> dividend(*this);