MarcoFalke
ad51e1372b
Merge #13774: Return void instead of bool for functions that cannot fail
d78a8dc3e8 Return void instead of bool for functions that cannot fail (practicalswift)
Pull request description:
Return `void` instead of `bool` for functions that cannot fail:
* `CBlockTreeDB::ReadReindexing(...)`
* `CChainState::ResetBlockFailureFlags(...)`
* `CTxMemPool::addUnchecked(...)`
* `CWallet::CommitTransaction(...)`
* `CWallet::LoadDestData(...)`
* `CWallet::LoadKeyMetadata(...)`
* `CWallet::LoadScriptMetadata(...)`
* `CWallet::LoadToWallet(...)`
* `CWallet::SetHDChain(...)`
* `CWallet::SetHDSeed(...)`
* `PendingWalletTx::commit(...)`
* `RemoveLocal(...)`
* `SetMinVersion(...)`
* `StartHTTPServer(...)`
* `StartRPC(...)`
* `TorControlConnection::Disconnect(...)`
Some of the functions can fail by throwing.
Found by manually inspecting the following candidate functions:
```
$ git grep -E '(^((static|virtual|inline|friend)[^a-z])*[^a-z]*bool [^=]*\(|return true|return false)' -- "*.cpp" "*.h"
```
Tree-SHA512: c0014e045362dbcd1a0cc8f69844e7b8cbae4f538e7632028daeca3a797ac11d8d3d86ebc480bedcb8626df3e96779d592747d52a12556fc49921b114fa0ccc6
2018-07-29 07:56:36 -04:00
..
2018-07-24 15:05:37 +09:00
2018-07-24 15:05:37 +09:00
2018-07-24 15:05:37 +09:00
2018-07-24 16:34:03 +02:00
2018-07-24 15:05:38 +09:00
2018-07-12 00:48:49 +02:00
2018-06-18 16:35:39 +01:00
2018-04-07 11:48:27 -05:00
2018-05-23 19:26:18 +02:00
2018-04-23 10:49:21 -04:00
2018-04-23 10:49:21 -04:00
2018-07-24 15:05:37 +09:00
2018-07-25 15:39:04 +02:00
2018-07-20 14:28:50 +02:00
2018-07-16 16:08:24 -07:00
2018-07-29 07:56:36 -04:00
2018-07-27 13:19:03 +02:00
2018-07-27 13:19:03 +02:00
2018-07-12 10:30:21 +01:00
2018-03-22 16:23:04 +01:00