In #2374 VartimeBasepointTable and the PointWithVartimeBasepointTable trait were added.
This was perhaps inspired by looking at curve25519-dalek which features a number of Vartime* prefixed traits. However in crypto-bigint we've already adopted a convention of putting *Vartime as a suffix, which matches nicely with *_vartime method names (e.g. #2286 added lincomb_vartime, and *_vartime is already the convention in crypto-bigint).
I would suggest we adopt a convention of *Vartime suffixes, e.g. MulVartime and MulByGeneratorVartime. I will change the existing names added in #2374 to suit.
In #2374
VartimeBasepointTableand thePointWithVartimeBasepointTabletrait were added.This was perhaps inspired by looking at
curve25519-dalekwhich features a number ofVartime*prefixed traits. However incrypto-bigintwe've already adopted a convention of putting*Vartimeas a suffix, which matches nicely with*_vartimemethod names (e.g. #2286 addedlincomb_vartime, and*_vartimeis already the convention incrypto-bigint).I would suggest we adopt a convention of
*Vartimesuffixes, e.g.MulVartimeandMulByGeneratorVartime. I will change the existing names added in #2374 to suit.