When merging GNU MP 6.2.0 mini-gmp.c into GNU Emacs (which uses mini-gmp on platforms lacking libgmp), I noticed three trivial fixes that had been applied on the Emacs copy that would be useful to propagate upstream to GNU MP. The first is trailing whitespace on a source code line (which the Emacs version-control system warns about). The other two are duplicated words "the the" and "to to", respectively. Please see the attached patch.
diff -r a615debc4e93 mini-gmp/mini-gmp.c
--- a/mini-gmp/mini-gmp.c	Fri Jan 24 04:38:57 2020 +0100
+++ b/mini-gmp/mini-gmp.c	Mon Jan 27 16:08:00 2020 -0800
@@ -790,7 +790,7 @@
 
        qh' = floor( (b^3 - 1) / u) - b = floor ((b^3 - b u - 1) / u
 	   = floor( (b (~u) + b-1) / u),
-	   
+
        and the remainder
 
        r = b (~u) + b-1 - qh (b uh + ul)
@@ -3418,7 +3418,7 @@
       gmp_lucas_step_k_2k (V, Qk, n);
 
       /* A step k->k+1 is performed if the bit in $n$ is 1	*/
-      /* mpz_tstbit(n,bs) or the the bit is 0 in $n$ but	*/
+      /* mpz_tstbit(n,bs) or the bit is 0 in $n$ but	*/
       /* should be 1 in $n+1$ (bs == b0)			*/
       if (b0 == bs || mpz_tstbit (n, bs))
 	{
@@ -4488,7 +4488,7 @@
       ptrdiff_t word_step;
       /* The current (partial) limb. */
       mp_limb_t limb;
-      /* The number of bytes left to to in this limb. */
+      /* The number of bytes left to do in this limb. */
       size_t bytes;
       /* The index where the limb was read. */
       mp_size_t i;
_______________________________________________
gmp-bugs mailing list
[email protected]
https://gmplib.org/mailman/listinfo/gmp-bugs

Reply via email to