On Mon, 2025-08-25 at 22:26 +0530, Nandakumar Edamana wrote: > On 25/08/25 21:54, Eduard Zingerman wrote: > > On Mon, 2025-08-25 at 09:46 +0530, Nandakumar Edamana wrote: > > > Status as of now: > > > > > > DECIDED: > > > > > > 1. Replace the current outer comment for the new tnum_mul() with a > > > cleaner explanation and the example from the README of the test > > > program. > > > > > > 2. (Related to PATCH 2/2) Drop the trivial tests. > > > > > > UNDECIDED: > > > > > > Instead of just doing tnum_mul(a, b), > > > > > > a) whether to do best(tnum_mul(a, b), tnum_mul(b, a)) > > > b) whether to do best(best(tnum_mul(a, b), tnum_mul(b, a)), > > > best(tnum_mul_old(a, b), tnum_mul_old(b, a))) > > I'd drop both undecided points. > Shall I send v5 with the decided changes then? Yes, please do.
