forked from bitcoin/bitcoin
-
Notifications
You must be signed in to change notification settings - Fork 2
New estimates #3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Closed
Closed
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
b9b1096 to
3e0c761
Compare
4f8aed0 to
56cc084
Compare
This makes it easier for us to replace it if desired, since it's now only in one spot. Also, it avoids the openssl include from allocators.h, which essentially forced openssl to be included from every compilation unit.
f5791c6 Add more information to errors in ReadBlockFromDisk (Wladimir J. van der Laan)
e5ece05 Rename Interval() to DifficultyAdjustmentInterval() (Shaul Kfir)
850c570 No longer check osx compatibility in RenameThread (Michael Ford)
544e64e Update debian changelog and control file (Matt Corallo)
- creates a "Bitcoin (testnet).lnk" when on testnet or a "Bitcoin (regtest).lnk, when on regtest - fixes bitcoin#5778
1630219 openssl: abstract out OPENSSL_cleanse (Cory Fields)
e5d9d77 fix crash: createmultisig and addmultisigaddress (fsb4000)
This reverts commit 1078fb0 (and thus pull bitcoin#5623). It has various issues: - Pull request names get cut off at ", see e.g. a026a56 - Merge script no longer copes with pulls that have a milestone attached, due to a duplicate 'title' in JSON that is not handled by the ad-hoc parsing.
Its original placement was causing confusion among some users.
Split GetNextWorkRequired() into two functions to allow the difficulty calculations to be tested without requiring a full blockchain. Add unit tests to cover basic difficulty calculation, plus each of the min/max actual time, and maximum difficulty target conditions.
b63ae56 Add list of implemented BIPs (Pieter Wuille)
This is really a packager's option. While it's helpful to encourage devs to test this option for daily builds, it's not reliable in several real-world use-cases. Some older libstdc++ runtimes (freebsd 9, debian wheezy, for example) fail to properly catch exceptions due to mismatched type_info. See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=19664 for more info.
Backwards-compatibility for libstdc++ is not limited to straightforward abi changes. Symbol visibility also needs to be taken into consideration, and that really can't be addressed simply. Instead, just static-link libstdc++ for backwards-compat.
…travis
For Gitian releases:
- Windows builds remain unchanged. libstdc++ was already linked statically.
- OSX builds remain unchanged. libstdc++ is tied to the SDK and not worth
messing with.
- Linux builds now statically link libstdc++.
For Travis:
- Match the previous behavior by adding --enable-reduce-exports as
necessary.
- Use static libstdc++ for the full Linux build.
With headers-first we can compare against the best header timestamp, rather than using checkpoints which require code updates to maintain.
Code to avoid calling Perfmon too often is only needed when perfmon is actually going to get called. This is not intended to make any functional difference in the addition of entropy to the random pool.
src/strlcpy.h was removed by bitcoin@6032e4f
- write "Bitcoins" uppercase - replace secure/insecure for payment requests with authenticated/unauthenticated - change a translatable string for payment request expiry to match another existing string to only get ONE resulting string to translate
Comment by pryds on Transifex.
- BITCOIN_FOLDER_SUBFOLDER_FILENAME_H
c6de7c3 trivial string change in wallet.cpp (the -> that) (Philip Kaufmann) 1e3473d Add operator names to DNS Seed list (Michael Ford) a21df62 ensure consistent header comment naming conventions (Philip Kaufmann) 9e16cb1 Make 'Default: %u' spacing consistent in help message (Wladimir J. van der Laan) 6cb37a3 [Qt, Trivial] remove two unneeded includes of wallet/db.h (Philip Kaufmann) 68f795e [Qt, Trivial] fix header groupings + space, intdentation fixes (Philip Kaufmann) 985ec17 [Qt, Trivial] misc minor string changes (Philip Kaufmann) c0555dc depends: latest config.guess and config.sub (Michael Ford) 89fc6b6 Remove leftover strlcpy.h copyright (Rob Van Mieghem) 468aa3b Re-wrote a passage of text that was difficult to understand. (Jason Lewicki) 71ad6bd [Trivial] format sync.h (Philip Kaufmann) abcec30 Update REST URL to match reality (paveljanik) 447d37e Use https link to bitcoin.org in Doxygen intro (Michael Ford) c069234 Fix typo in init.cpp interpration/interpretation (Michael Ford) 52070c8 Removed '()' where used without contents inside (Nicolas Benoit) 30c1db1 Replaced current function names with __func__ in LogPrintf() calls. (Nicolas Benoit) 9bdd03f Point to the Debian 7.8 installer (Michael Ford) 0b2f930 Fix docs for 'complete' field in 'signrawtransaction' response (charlescharles) c2f2161 Add x86_64* i686* mips* and arm* to depends .gitignore (Michael Ford) fa535f9 Remove folder and images for bootstrap.md (sandakersmann) 60c1469 [Qt] header group cleanup (Philip Kaufmann)
c1ecee8 Set nSequenceId when a block is fully linked (Suhas Daftuar)
a784f90 Cap nAttempts penalty at 8 and switch to pow instead of a division loop. (Gregory Maxwell)
Add test for -proxy, -onion and -proxyrandomize.
This class groups transactions that have been confirmed in blocks into buckets, based on either their fee or their priority. Then for each bucket, the class calculates what percentage of the transactions were confirmed within various numbers of blocks. It does this by keeping an exponentially decaying moving history for each bucket and confirm block count of the percentage of transactions in that bucket that were confirmed within that number of blocks. Also eliminate txs which weren't clear at entry from fee/pri calcs Add dynamic breakpoints and tracking of confirmation delays in mempool transactions Remove old CMinerPolicyEstimator and CBlockAverage code New smartfees.py
Pass a flag to the estimation code, using IsInitialBlockDownload as a proxy for when we are still catching up and we shouldn't be counting how many blocks it takes for transactions to be included. Also rearrange HasNoInputs so TxMemPoolEntrys don't need to be aware of TxMempools.
SQUASHME: TRIVIAL: Decouple CBlockPolicyEstimator from boost/foreach SQUASHME: TRIVIAL: Remove unused minRelayFee param from CBlockPolicyEstimator::Read() SQUASHME: Decouple CBlockPolicyEstimator from minRelayTxFee global Add a hard coded minimum feerate and priority for the estimator buckets
morcos
pushed a commit
that referenced
this pull request
Oct 22, 2016
16a1f7f Merge #3: Pull upstream 3f03bfd Merge pull request bitcoin#27 from laanwj/2016_09_const_refs 5668ca3 Return const references from getKeys, getValues, get_str cedda14 Merge pull request bitcoin#28 from MarcoFalke/patch-1 9f0b997 [travis] Work around osx libtool issue git-subtree-dir: src/univalue git-subtree-split: 16a1f7f6e9ed932bec7c7855003affea1e165fb5
morcos
pushed a commit
that referenced
this pull request
Oct 23, 2017
fe805ea74 Declare single-argument (non-converting) constructors "explicit" 8a2d6f1e3 Merge pull request bitcoin#41 from jgarzik/get-obj-map ba341a20d Add getObjMap() helper method. Also, constify checkObject(). ceb119413 Handle .pushKV() and .checkObject() edge cases. 107db9829 Add ::push_back(double) method for feature parity. d41530031 Move one-line implementation of UniValue::read() to header. 52e85b35b Move exception-throwing get_* methods into separate implementation module. dac529675 README.md: update code quotes 3e31dcffb README.md: close code quote d09b8429d Update README.md f1b86edb4 Convert README to markdown style. 1dfe464ef Import UniValue class unit tests from bitcoin project. 0d3e74dd1 operator[] takes size_t index parameter (versus unsigned int) 640158fa2 Private findKey() method becomes size_t clean, and returns bool on failure. 709913585 Merge pull request bitcoin#36 from ryanofsky/pr/end-str a31231b51 Version 1.0.3 4fd5444d1 Reject unterminated strings 81eba332b Merge pull request bitcoin#26 from isle2983/pushBackHelpers 36405413e Merge PR bitcoin#32 from branch 'nul-not-special' of git://github.com/ryanofsky/univalue into merge 89bb07322 Merge pull request bitcoin#31 from ryanofsky/raw-literals 511008c36 Merge pull request bitcoin#30 from ryanofsky/test-driver 77974f3a9 Merge pull request bitcoin#34 from paveljanik/20161116_Wshadow_codepoint a38fcd355 Do not shadow member variable codepoint. fd32d1ab8 Don't require nul-terminated string inputs 0bb1439d0 Support parsing raw literals in UniValue 28876d045 Merge pull request bitcoin#29 from btcdrak/exportspace 839ccd71f Add test driver for JSONTestSuite 26ef3fff1 Remove trailing whitespace from JSON export cfa0384d6 Convenience wrappers for push_back-ing integer types REVERT: 16a1f7f6e Merge #3: Pull upstream REVERT: daf1285af Merge pull request #2 from jgarzik/master REVERT: f32df99e9 Merge branch '2016_04_unicode' into bitcoin REVERT: 280b191cb Merge remote-tracking branch 'jgarzik/master' into bitcoin REVERT: 2740c4f71 Merge branch '2015_11_escape_plan' into bitcoin git-subtree-dir: src/univalue git-subtree-split: fe805ea74f8919382720b09a905a14e81311b3ad
morcos
pushed a commit
that referenced
this pull request
Jan 16, 2018
…tion d31e5c1 Fix shutdown in case of errors during initialization (Wladimir J. van der Laan) Pull request description: PR bitcoin#10286 introduced a few steps which are not robust to early shutdown in initialization. Stumbled upon this with bitcoin#11781, not sure if there are other scenarios that can trigger it, but it's good to harden against this in any case. E.g. ``` $ src/bitcoind -debuglogfile=/dfdf Error: Could not open debug log file /dfdf Program received signal SIGSEGV, Segmentation fault. UnregisterValidationInterface (pwalletIn=0x0) at /.../bitcoin/src/validationinterface.cpp:82 82 g_signals.m_internals->BlockChecked.disconnect(boost::bind(&CValidationInterface::BlockChecked, pwalletIn, _1, _2)); (gdb) bt #0 UnregisterValidationInterface (pwalletIn=0x0) at /.../bitcoin/src/validationinterface.cpp:82 #1 0x00005555555a11fc in Shutdown () at /.../bitcoin/src/init.cpp:196 #2 0x00005555555961cc in AppInit (argc=<optimized out>, argv=<optimized out>) at /.../bitcoin/src/bitcoind.cpp:183 #3 0x0000555555596249 in main (argc=0, argv=0x555555ecf200) at /.../bitcoin/src/bitcoind.cpp:19 ``` Tree-SHA512: 7dd9570a9803514a17781bfadf1edde47e96df4e852cce2f423cab422e005fb94d44e777af1a6ea5167b04a4d889e848ae7a61a7e0e94232247ddea32ee70fc8
morcos
pushed a commit
that referenced
this pull request
Jan 16, 2018
07947ff2da Merge bitcoin#9: [tests] Fix BOOST_CHECK_THROW macro ec849d9a28 [tests] Fix BOOST_CHECK_THROW macro 31bc9f5a49 Merge bitcoin#8: Remove unused Homebrew workaround fa042093d1 Remove HomeBrew workaround a523e08ae4 Merge #7: Declare single-argument (non-converting) constructors "explicit" a9e53b38ba Merge #4: Pull upstream 16a1f7f6e9 Merge #3: Pull upstream daf1285af6 Merge pull request #2 from jgarzik/master f32df99e96 Merge branch '2016_04_unicode' into bitcoin 280b191cb1 Merge remote-tracking branch 'jgarzik/master' into bitcoin 2740c4f712 Merge branch '2015_11_escape_plan' into bitcoin git-subtree-dir: src/univalue git-subtree-split: 07947ff2da9ef02a9dfa13346bc5545708e3ebe7
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.