[Wallet] Move IncOrderPosNext to CWallet::AddAccountingEntry #8585

pull pstratem wants to merge 1 commits into bitcoin:master from pstratem:2016-08-24-cwallet-incorderposnext changing 3 files +9 −8
  1. pstratem commented at 4:07 AM on August 25, 2016: contributor

    No description provided.

  2. pstratem force-pushed on Aug 25, 2016
  3. pstratem renamed this:
    Remove last caller of IncOrderPosNext
    [Wallet] Move IncOrderPosNext to CWallet::AddAccountingEntry
    on Aug 25, 2016
  4. pstratem force-pushed on Aug 25, 2016
  5. pstratem force-pushed on Aug 25, 2016
  6. pstratem force-pushed on Aug 25, 2016
  7. Move IncOrderPosNext to CWallet::AddAccountingEntry ac68d20322
  8. pstratem force-pushed on Aug 25, 2016
  9. jonasschnelli added the label Wallet on Aug 25, 2016
  10. jonasschnelli commented at 6:30 AM on August 25, 2016: contributor

    Concept ACK. Accounting tests (unit tests) are failing.

  11. in src/wallet/wallet.h:None in ac68d20322 
     738 | @@ -737,7 +739,6 @@ class CWallet : public CCryptoKeyStore, public CValidationInterface
 739 |       * Increment the next transaction order id
    MarcoFalke commented at 8:52 AM on August 25, 2016:

    Could you move the comment as well?

  12. BangkitSedar commented at 9:29 AM on August 25, 2016: none

    Try new pull

  13. pstratem commented at 12:13 AM on September 17, 2016: contributor

    replaced by #8696

  14. pstratem closed this on Sep 17, 2016
  15. MarcoFalke locked this on Sep 8, 2021
Contributors
pstratemjonasschnelliMarcoFalkeBangkitSedar
Labels
Wallet
github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2026-04-19 00:15 UTC
This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me