Skip to content

[wip] Arrange about CChainParams. - #27

Open
kobake wants to merge 6 commits into
master-tmpfrom
arrange-chain-params
Open

[wip] Arrange about CChainParams.#27
kobake wants to merge 6 commits into
master-tmpfrom
arrange-chain-params

Conversation

@kobake

@kobake kobake commented May 18, 2017

Copy link
Copy Markdown
Owner

master の wip 解消用。一旦 master-tmp へのマージが完了したら master-tmp の内容で master のほうに force push する。

対応内容:CMainParams, CTestNetParams, CRegTestParams のグローバル変数を定義しているが、それのコンストラクタが何故か呼ばれないことがあり、それの対症療法として変数への参照を適当に定義した(ものすごく気持ち悪い)

現在の状況:試行錯誤したコミットを wip として記録しているのであとでちゃんと squash する。tmp って付いてるコミットは不要(検証中のビルド時間を短くするためにテストコードビルドを省くコミットを入れた)なので rebase で消す予定。

一応現段階では問題は解決したが、気持ち悪い対症療法は嫌なのでもう少し調査を進めて根本的な解決を探りたい。理想をいえばそれが解決してから squash とマージを行う。調査の結果、どうにもならなかったら対症療法のまま進めてしまう。

@kobake
kobake force-pushed the arrange-chain-params branch from f32c8de to 8630d4c Compare May 18, 2017 12:19
@kobake
kobake force-pushed the arrange-chain-params branch from 8630d4c to 435d539 Compare May 18, 2017 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant