Member AcceptableInputs (CTxMemPool &pool, CValidationState &state, const CTransaction &tx, bool fLimitFree, bool *pfMissingInputs, bool fRejectInsaneFee=false, bool isDSTX=false)

: @campv need to recompute the fee

Member AcceptableInputs (CTxMemPool &pool, CValidationState &state, const CTransaction &tx, bool fLimitFree, bool *pfMissingInputs, bool fRejectInsaneFee=false, bool isDSTX=false)

: @campv need to recompute the fee

Member AcceptBlock (CBlock &block, CValidationState &state, CBlockIndex **pindex, CDiskBlockPos *dbp=NULL, bool fAlreadyCheckedBlock=false)

: deal better with duplicate blocks.

: Remove this chain from disk.

Member AcceptBlock (CBlock &block, CValidationState &state, CBlockIndex **pindex, CDiskBlockPos *dbp=NULL, bool fAlreadyCheckedBlock=false)

: deal better with duplicate blocks.

: Remove this chain from disk.

Member AcceptBlockHeader (const CBlock &block, CValidationState &state, CBlockIndex **ppindex)

: ENABLE BLOCK CACHE IN SPECIFIC CASES

Member AllocateFileRange (FILE *file, unsigned int offset, unsigned int length)

: just write one byte per block

Member AppInit2 (bool isDaemon)

*** do we still need this after -printtoconsole is gone?

Member AskPassphraseDialog::accept ()

: get rid of this .c_str() by implementing SecureString::operator=(std::string)

Member BackupWallet (const CWallet &wallet, const fs::path &strDest, bool fEnableCustom=true)

: add seconds to avoid naming conflicts

Member BackupWallet (const CWallet &wallet, const fs::path &strDest, bool fEnableCustom=true)

: add seconds to avoid naming conflicts

Member BitcoinUnits::formatWithUnit (int unit, const CAmount &amount, bool plussign=false, SeparatorStyle separators=separatorStandard)

: Review all remaining calls to BitcoinUnits::formatWithUnit to

: determine whether the output is used in a plain text context

: or an HTML context (and replace with

: BtcoinUnits::formatHtmlWithUnit in the latter case).

: either context.

: there aren't instances where the result could be used in

Member CAddress::nServices

: make private (improves encapsulation)

Member CAddrMan::Good_ (const CService &addr, int64_t nTime)

: maybe re-add the node, but for now, just bail out

Member CAddrMan::size ()

: Cache this in an atomic to avoid this overhead

Member CallRPC (const std::string &strMethod, const UniValue &params)

RAII

Member CBlock::IsProofOfAudit () const

add condition check for Audit mining

Member CBudgetProposal::IsValid (std::string &strError, bool fCheckCollateral=true)

: We should keep track of the last time the proposal was valid, if it's invalid for 2 weeks, erase it

Member CCryptoKeyStore::vMasterKey

: In the future, move this variable to the wallet class directly following upstream's structure.

Member CECKey::SanityCheck ()

Is there more EC functionality that could be missing?

Member CheckBlock (const CBlock &block, CValidationState &state, bool fCheckPOW=true, bool fCheckMerkleRoot=true, bool fCheckSig=true)

Audit checkblock

Member CheckBlock (const CBlock &block, CValidationState &state, bool fCheckPOW=true, bool fCheckMerkleRoot=true, bool fCheckSig=true)

Audit checkblock

Member CInv::type

: make private (improves encapsulation)

Member CMasternode::IsValidNetAddr ()

: regtest is fine with any addresses for now,

Member CMasternodeMan::nDsqCount

: Remove this from serialization

Member CMasternodePing::CheckAndUpdate (int &nDos, bool fRequireEnabled=true, bool fCheckSigTimeOnly=false)

: or should we also request this block?

Member CNode::AbortMessage () UNLOCK_FUNCTION(cs_vSend)

: Document the precondition of this function. Is cs_vSend locked?

Member CNode::BeginMessage (const char *pszCommand) EXCLUSIVE_LOCK_FUNCTION(cs_vSend)

: Document the postcondition of this function. Is cs_vSend locked?

Member CNode::EndMessage () UNLOCK_FUNCTION(cs_vSend)

: Document the precondition of this function. Is cs_vSend locked?

Member CoinControlDialog::viewItemChanged (QTreeWidgetItem *, int)

: Remove this temporary qt5 fix after Qt5.3 and Qt5.4 are no longer used.

Member CONSOLE_HISTORY

: add a scrollback limit, as there is currently none

: make it possible to filter out categories (esp debug messages when implemented)

: receive errors and debug messages through ClientModel

Member ConvertBitsToDouble (unsigned int nBits)

***

Member CWallet::CreateTransaction (const std::vector< std::pair< CScript, CAmount > > &vecSend, CWalletTx &wtxNew, CReserveKey &reservekey, CAmount &nFeeRet, std::string &strFailReason, const CCoinControl *coinControl=NULL, AvailableCoinsType coin_type=ALL_COINS, bool useIX=false, CAmount nFeePay=0)

: pass in scriptChange instead of reservekey so

Member CWalletKey::CWalletKey (int64_t nExpires=0)

: add something to note what created it (user, getnewaddress, change) maybe should have a map<string, string> property map

Member encryptwallet (const UniValue &params, bool fHelp)

: get rid of this .c_str() by implementing SecureString::operator=(std::string)

Member getblocktemplate (const UniValue &params, bool fHelp)

: Maybe recheck connections/IBD and (if something wrong) send an expires-immediately template to stop miners?

Member gettxout (const UniValue &params, bool fHelp)

: this should be done by the CCoinsViewMemPool

Member limitedmap< K, V >::update (const_iterator itIn, const mapped_type &v)

: When we switch to C++11, use map.erase(itIn, itIn) to get the non-const iterator.

Member MAX_AMOUNT_LOADED_RECORDS

, add load on demand in pages (not every tx loaded all the time into the records list).

Member OptionsDialog::showRestartWarning (bool fPersistent=false)

: should perhaps be a class attribute, if we extend the use of statusLabel

Member org.bitcoin.NativeSecp256k1.computePubkey (byte[] seckey)

add a 'compressed' arg

Member org.bitcoin.NativeSecp256k1Test.testVerifyPos ()

improve comments/add more tests

Member OverviewPage::setBalance (const CAmount &balance, const CAmount &unconfirmedBalance, const CAmount &immatureBalance, const CAmount &watchOnlyBalance, const CAmount &watchUnconfBalance, const CAmount &watchImmatureBalance)