mirror of https://github.com/oxen-io/oxen-core.git
6aa9db9538
- Remove implicit `operator bool` from ec_point/public_key/etc. which
was causing all sorts of implicit conversion mess and bugs.
- Change ec_point/public_key/etc. to use a `std::array<unsigned char,
32>` (via a base type) rather than a C-array of char that has to be
reinterpret_cast<>'ed all over the place.
- Add methods to ec_point/public_key/etc. that make it work more like a
container of bytes (`.data()`, `.size()`, `operator[]`, `begin()`,
`end()`).
- Make a generic `crypto::null<T>` that is a constexpr all-0 `T`, rather
than the mishmash `crypto::null_hash`, crypto::null_pkey,
crypto:#️⃣:null(), and so on.
- Replace three metric tons of `crypto::hash blahblah =
crypto::null_hash;` with the much simpler `crypto::hash blahblah{};`,
because there's no need to make a copy of a null hash in all these
cases. (Likewise for a few other null_whatevers).
- Remove a whole bunch of `if (blahblah == crypto::null_hash)` and `if
(blahblah != crypto::null_hash)` with the more concise `if
(!blahblah)` and `if (blahblah)` (which are fine via the newly
*explicit* bool conversion operators).
- `crypto::signature` becomes a 64-byte container (as above) but with
`c()` and `r()` to get the c() and r() data pointers. (Previously
`.c` and `.r` were `ec_scalar`s).
- Delete with great prejudice CRYPTO_MAKE_COMPARABLE and
CRYPTO_MAKE_HASHABLE and all the other utter trash in
`crypto/generic-ops.h`.
- De-inline functions in very common crypto/*.h files so that they don't
have to get compiled 300 times.
- Remove the disgusting include-a-C-header-inside-a-C++-namespace
garbage from some crypto headers trying to be both a C and *different*
C++ header at once.
- Remove the toxic, disgusting, shameful `operator&` on ec_scalar, etc.
that replace `&x` with `reinterpret_cast x into an unsigned char*`.
This was pure toxic waste.
- changed some `<<` outputs to fmt
- Random other small changes encountered while fixing everything that
cascaded out of the above changes.
|
||
|---|---|---|
| .. | ||
| CMakeLists.txt | ||
| block_reward.cpp | ||
| block_reward.h | ||
| block_validation.cpp | ||
| block_validation.h | ||
| bulletproofs.cpp | ||
| bulletproofs.h | ||
| chain_split_1.cpp | ||
| chain_split_1.h | ||
| chain_switch_1.cpp | ||
| chain_switch_1.h | ||
| chaingen.cpp | ||
| chaingen.h | ||
| chaingen001.cpp | ||
| chaingen_main.cpp | ||
| chaingen_tests_list.h | ||
| double_spend.cpp | ||
| double_spend.h | ||
| double_spend.inl | ||
| integer_overflow.cpp | ||
| integer_overflow.h | ||
| multisig.cpp | ||
| multisig.h | ||
| oxen_tests.cpp | ||
| oxen_tests.h | ||
| rct.cpp | ||
| rct.h | ||
| ring_signature_1.cpp | ||
| ring_signature_1.h | ||
| transaction_tests.cpp | ||
| transaction_tests.h | ||
| tx_validation.cpp | ||
| tx_validation.h | ||
| v2_tests.cpp | ||
| v2_tests.h | ||
| wallet_tools.cpp | ||
| wallet_tools.h | ||