ByteReverse(またはその不在)に関して何か奇妙なことが起こっていると思う。「data」と「ナンス」をバイトリバースする必要があるかどうか、またどのようにするかがまったく不明確だ。
getworkがバイト反転を行う。midstate、data、hash1はすでにビッグエンディアンで、dataを返す時もビッグエンディアンのままなので、ビッグエンディアンで作業し、バイト反転は不要だ。これらはScanHash_関数に渡されるのと同じデータだ。midstate、data、hash1を16バイトアラインされたバッファに入れてScanHash_関数に渡すことができる。例えばScanHash(pmidstate, pdata + 64, phash1, nHashesDone)のように。ナンスが見つかったらdataにパッチしてgetworkを呼び出す。
ScanHash_関数がpdata + 64ではなくpdataを使うように変更して一貫性を持たせるべきかもしれない。
targetはリトルエンディアンで、m0mchilのものと同じ方法のはずだ。(もし違えば修正すべきだ)バイト反転を使う唯一のケースだ。こんな感じだと思う:if ByteReverse((unsigned int*)hash[6]) < (unsigned int*)target[6]。
DiabloD3の投稿(2010年11月24日11:31 UTC)サトシ、getworkの実装をm0mchilの仕様に準拠するように修正してほしい
これが新しい仕様だ。マイナーをこれに対応させるのは難しくないはずだ。
変更点:
- 候補を送信した時にworkを返さない。パラメーターなしで呼び出した時のみだ。
- blockフィールドがdataとhash1に分割された。
- 一貫性のためstateをmidstateに改名。
- extranonceは不要になった。