-
Notifications
You must be signed in to change notification settings - Fork 206
Get rid of BN prefix #284
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Get rid of BN prefix #284
Conversation
b375520
to
11739a1
Compare
0ac108d
to
d2c5e30
Compare
fd4751a
to
6cc5295
Compare
@sjaeckel Any reason to delay this PR for 2.0 (in general, what are you plans about a new release, removing deprecations etc)? This PR shouldn't have any user facing implications (no abi/api changes). |
it changes the user facing defines in |
tommath_class.h is not included in tommath.h anymore. |
that's true, but I still count this way of tweaking/configuring ltm as a feature that I don't want to break. |
ed27ff8
to
dcf5738
Compare
4e95c38
to
2b8d947
Compare
780d866
to
64b1148
Compare
1697fd7
to
1dd9eb5
Compare
* git blame <renamed-file> is not affected * git log --follow <renamed-file> can be used to show log across renames
@sjaeckel could you please merge this soon since otherwise we have to chaotically rebase everything very often. @czurnieden is already starting with new stuff. |
I wanted to merge yesterday already but had to leave, sorry |
Another trivial one 😁
In the beginning it seemed there were three different prefixes, MP, BN and LTM. Fortunately it seems we now settled on mp and s_mp. This renames all the things to comply with that.
Furthermore I find the current
bn_s_mp_
prefix a bit hard on the eye.This is done similarly to #172 via a move script.
Not to be merged now, don't worry about all your open PRs!