Skip to content
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

Add missing breaks in optionmodel's switch case #1118

Merged
merged 1 commit into from
Apr 17, 2012

Conversation

sipa
Copy link
Member

@sipa sipa commented Apr 17, 2012

No description provided.

@laanwj
Copy link
Member

laanwj commented Apr 17, 2012

ACK

laanwj added a commit that referenced this pull request Apr 17, 2012
Add missing breaks in optionmodel's switch case
@laanwj laanwj merged commit ac4161e into bitcoin:master Apr 17, 2012
coblee pushed a commit to litecoin-project/litecoin that referenced this pull request Jul 17, 2012
Add missing breaks in optionmodel's switch case
@sipa sipa deleted the addbreaks branch June 23, 2017 00:42
lateminer pushed a commit to lateminer/bitcoin that referenced this pull request Jan 22, 2019
* Clarify variable name nCoinCacheUsage

This name implies the current dynamic usage which is incorrect it is
actually the current max size of teh coins cache: change the name
to nCoinCacheMaxSize.

* Fix bug in correcting drift of cachedCoinsUsage

Becuase of the confusing naming of nCoinsCacheUsage a bug was
introduced which assumed that nCoinsCacheUsage was the current
usage of the coins cache rather than it's max size.  By changing
the naming of the variables the bug is now obvious and we can reset
the coins cache when/if the nCurrentCacheUsage falls below zero.

* replace nCoinCacheUsage definition with nCoinCacheMaxSize
lateminer pushed a commit to lateminer/bitcoin that referenced this pull request Dec 25, 2019
…Height() methods call.

aee39f6 [Spork] Guard chainActive.Tip() and chainActive.Height() methods call. (furszy)

Pull request description:

  Auto explainable title, guarding chainActive calls.

ACKs for top commit:
  random-zebra:
    utACK aee39f6
  Warrows:
    utACK aee39f6

Tree-SHA512: 6c9e82746dc21e57de3276cc1b568302d037b7eb3e2da8ca5cc97babd95f0d2871eaa2789496c05820d01a4123be9e49ff96036eb2959ed53b164162892aa1ec
Bushstar pushed a commit to Bushstar/omnicore that referenced this pull request May 2, 2020
…nputs are missing

56a05fe Add error text for omni_decodetransaction, when inputs are missing (dexX7)

Pull request description:

  When the node is not synchronized, transaction inputs need to be provided explicitly. When this is not the case, a proper error is now shown:

  > "Transaction inputs were not found. Please provide inputs explicitly (see help description) or fully synchronize node."

  It resolves bitcoin#1110.

Tree-SHA512: d2de932e423065a22c0bde73ba148abbd2d323ff3a1fe92fcb48a3b30f5ae8eb47d51867d88b0f12e7692e054809824560d9bfc38d5a537f7ea1db30206491f8
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants