<ol>
<li>Yeah, I think you're right that the transcationConfidence fixes are no longer needed if the txs are never shared by two wallets.</li>
<li>Indeed, it's probably better to do the cloning in Wallet.receicePending() to guarantee that no tx is in multiple wallets. For the Wallet.receiveFromBlock() I notice it's already cloned before being added, see AbstractBlockChain:688 so I think the intention is that no tx should be shared between wallets but the internally shared for pending transactions case wasn't handled and not many people have been running with shared wallets like this.</li>
</ol>

<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly, <a href="https://github.com/bisq-network/bitcoinj/pull/18#issuecomment-449973919">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AkpZtiR_7wdN_g1dQR9U7P2F0C2gKbcBks5u84dJgaJpZM4Zgibw">mute the thread</a>.<img src="https://github.com/notifications/beacon/AkpZtvKtybOKToZGD63FOsLO0LVRGyEeks5u84dJgaJpZM4Zgibw.gif" height="1" width="1" alt="" /></p>
<script type="application/json" data-scope="inboxmarkup">{"api_version":"1.0","publisher":{"api_key":"05dde50f1d1a384dd78767c55493e4bb","name":"GitHub"},"entity":{"external_key":"github/bisq-network/bitcoinj","title":"bisq-network/bitcoinj","subtitle":"GitHub repository","main_image_url":"https://github.githubassets.com/images/email/message_cards/header.png","avatar_image_url":"https://github.githubassets.com/images/email/message_cards/avatar.png","action":{"name":"Open in GitHub","url":"https://github.com/bisq-network/bitcoinj"}},"updates":{"snippets":[{"icon":"PERSON","message":"@sqrrm in #18: 1. Yeah, I think you're right that the transcationConfidence fixes are no longer needed if the txs are never shared by two wallets.\r\n2. Indeed, it's probably better to do the cloning in Wallet.receicePending() to guarantee that no tx is in multiple wallets. For the Wallet.receiveFromBlock() I notice it's already cloned before being added, see AbstractBlockChain:688 so I think the intention is that no tx should be shared between wallets but the internally shared for pending transactions case wasn't handled and not many people have been running with shared wallets like this."}],"action":{"name":"View Pull Request","url":"https://github.com/bisq-network/bitcoinj/pull/18#issuecomment-449973919"}}}</script>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/bisq-network/bitcoinj/pull/18#issuecomment-449973919",
"url": "https://github.com/bisq-network/bitcoinj/pull/18#issuecomment-449973919",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>