diff options
| author | Wladimir J. van der Laan <[email protected]> | 2020-02-13 18:43:23 +0100 |
|---|---|---|
| committer | Wladimir J. van der Laan <[email protected]> | 2020-02-13 18:50:02 +0100 |
| commit | 470664f2b788e2cce9202991d11476a6fef00ef4 (patch) | |
| tree | 0dc207b063623014f2c13964e19b95c1ef853cbf /src/httpserver.cpp | |
| parent | Merge #18121: gui: Throttle GUI update pace when -reindex (diff) | |
| parent | refactor: rpc: Remove vector copy from listtransactions (diff) | |
| download | discoin-470664f2b788e2cce9202991d11476a6fef00ef4.tar.xz discoin-470664f2b788e2cce9202991d11476a6fef00ef4.zip | |
Merge #17746: refactor: rpc: Remove vector copy from listtransactions
25bc17fceb08ee9625c5e09e2579117ec6f7a1c5 refactor: rpc: Remove vector copy from listtransactions (João Barbosa)
Pull request description:
Current approach
- copy accumulated `ret` vector to `arrTmp`
- drop unnecessary elements from `arrTmp`
- reverse `arrTmp`
- clear `ret`
- copy `arrTmp` to the `ret`
New approach
- create a vector from the accumulated `ret` with just the necessary elements already reversed
- copy it to the result
This PR doesn't change behavior.
ACKs for top commit:
ryanofsky:
Code review ACK 25bc17fceb08ee9625c5e09e2579117ec6f7a1c5. Just comment and commit message tweaks since last review
Tree-SHA512: 87906561e3accdbdb0f4a8194cbcd76ea53ae53d0ce135b90bc54a5f77e300b14ef08505e7daf1fe52426f135442a743da5a027416a769bd454922357cebe7c0
Diffstat (limited to 'src/httpserver.cpp')
0 files changed, 0 insertions, 0 deletions