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 | ||
| account.cpp | ||
| aligned.cpp | ||
| apply_permutation.cpp | ||
| base58.cpp | ||
| block_queue.cpp | ||
| block_reward.cpp | ||
| blockchain_db.cpp | ||
| bulletproofs.cpp | ||
| chacha.cpp | ||
| checkpoints.cpp | ||
| command_line.cpp | ||
| crypto.cpp | ||
| device.cpp | ||
| difficulty.cpp | ||
| epee_boosted_tcp_server.cpp | ||
| epee_levin_protocol_handler_async.cpp | ||
| epee_utils.cpp | ||
| expect.cpp | ||
| get_xtype_from_string.cpp | ||
| hashchain.cpp | ||
| hmac_keccak.cpp | ||
| keccak.cpp | ||
| levin.cpp | ||
| lmdb.cpp | ||
| logging.cpp | ||
| long_term_block_weight.cpp | ||
| main.cpp | ||
| median.cpp | ||
| memwipe.cpp | ||
| mlocker.cpp | ||
| mnemonics.cpp | ||
| mul_div.cpp | ||
| multiexp.cpp | ||
| multisig.cpp | ||
| net.cpp | ||
| node_server.cpp | ||
| notify.cpp | ||
| output_distribution.cpp | ||
| output_selection.cpp | ||
| oxen_name_system.cpp | ||
| parse_address.cpp | ||
| parse_amount.cpp | ||
| pruning.cpp | ||
| random.cpp | ||
| random_path.h | ||
| ringct.cpp | ||
| ringdb.cpp | ||
| rolling_median.cpp | ||
| serialization.cpp | ||
| service_nodes.cpp | ||
| service_nodes_swarm.cpp | ||
| sha256.cpp | ||
| sqlite.cpp | ||
| string_util.cpp | ||
| subaddress.cpp | ||
| test_notifier.cpp | ||
| test_peerlist.cpp | ||
| test_protocol_pack.cpp | ||
| test_tx_utils.cpp | ||
| testdb.h | ||
| threadpool.cpp | ||
| unbound.cpp | ||
| unit_tests_utils.h | ||
| uri.cpp | ||
| varint.cpp | ||
| vercmp.cpp | ||
| wipeable_string.cpp | ||