Improved efficiency in constructors of COutPoint #10277

1. 
   mm-s commented at 2:18 pm on April 25, 2017: contributor

   the default constructor was executing memset(0) twice, first in uint256 default constructor and later in the call to SetNull.
2. laanwj added the label Resource usage on Apr 25, 2017
3. 
   jonasschnelli commented at 7:32 pm on April 25, 2017: contributor

   Looks good to me. But is there a reason to move the implementation away from the header?
4. 
   sipa commented at 11:22 pm on April 25, 2017: member

   Agree with @jonasschnelli. I expect that removing the ability to inline the constructor will have a worse effect than the double memset had.
5. 
   mm-s commented at 7:13 am on April 26, 2017: contributor

   I had no strong reason to move the impl, I moved it back to the header..
6. 
   laanwj commented at 10:19 am on April 26, 2017: member

   utACK after squash
7. 
   mm-s commented at 12:14 pm on April 27, 2017: contributor

   squashed, any further action required from my side?, thanks
8. 
   fanquake commented at 12:40 pm on April 27, 2017: member

   @mm-s Your squashing has introduced unrelated commits. Can please remove them, as well as the merge commit.
9. 
   MarcoFalke commented at 12:41 pm on April 27, 2017: member

   Still needs a rebase, maybe try this?:

   git fetch bitcoin git merge bitcoin master git reset –soft bitcoin/master git commit -m ’ Improved efficiency in constructors of COutPoint' git push origin master -f

   On Thu, Apr 27, 2017 at 2:14 PM, Marcos Mayorga notifications@github.com wrote:

   any action required from my side?, thanks

   — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub, or mute the thread.

10. 
    mm-s commented at 1:26 pm on April 27, 2017: contributor

    I don’t know how to remove merge commit 89b5b8d without getting errors when pushing (the tip of your current branch is behind) help on how to do it is welcome,… cannot it be merged in the current state?
11. mm-s closed this on Apr 28, 2017

    ---

12. mm-s force-pushed on Apr 28, 2017
13. MarcoFalke locked this on Sep 8, 2021

Contributors
mm-s jonasschnelli sipa laanwj fanquake MarcoFalke

Labels
Resource usage