<p></p>
<p><b>@chimp1984</b> requested changes on this pull request.</p>

<hr>

<p>In <a href="https://github.com/bisq-network/bisq/pull/4568#discussion_r499760038">core/src/main/java/bisq/core/btc/wallet/BtcWalletService.java</a>:</p>
<pre style='color:#555'>> @@ -584,23 +584,13 @@ public AddressEntry getOrCreateAddressEntry(String offerId, AddressEntry.Context
             if (emptyAvailableAddressEntry.isPresent()) {
                 return addressEntryList.swapAvailableToAddressEntryWithOfferId(emptyAvailableAddressEntry.get(), context, offerId);
             } else {
-                AddressEntry entry = new AddressEntry(wallet.freshReceiveKey(), context, offerId);
+                AddressEntry entry = new AddressEntry(wallet.freshReceiveKey(), context, offerId, true);
</pre>
<p>Is that never called from a trade protocol entry? e.g. the one for the multisig.</p>

<hr>

<p>In <a href="https://github.com/bisq-network/bisq/pull/4568#discussion_r499763395">core/src/main/java/bisq/core/btc/wallet/TradeWalletService.java</a>:</p>
<pre style='color:#555'>> @@ -601,6 +603,9 @@ public Transaction takerSignsDepositTx(boolean takerIsSeller,
             for (int i = 0; i < buyerInputs.size(); i++) {
                 TransactionInput transactionInput = makersDepositTx.getInputs().get(i);
                 depositTx.addInput(getTransactionInput(depositTx, getMakersScriptSigProgram(transactionInput), buyerInputs.get(i)));
+                if (!TransactionWitness.EMPTY.equals(transactionInput.getWitness())) {
+                    depositTx.getInput(depositTx.getInputs().size()-1).setWitness(transactionInput.getWitness());
</pre>
<p>Instead of <code>depositTx.getInputs().size()-1)</code> I would suggest to make a local var from the <code>getTransactionInput(depositTx, getMakersScriptSigProgram(transactionInput), buyerInputs.get(i))</code> we add earlier so its more clear. Maybe even do the check before adding, technically no difference but somehow more clean if the input gets completely defined before added.</p>

<hr>

<p>In <a href="https://github.com/bisq-network/bisq/pull/4568#discussion_r499765045">core/src/main/java/bisq/core/btc/wallet/TradeWalletService.java</a>:</p>
<pre style='color:#555'>> @@ -601,6 +603,9 @@ public Transaction takerSignsDepositTx(boolean takerIsSeller,
             for (int i = 0; i < buyerInputs.size(); i++) {
                 TransactionInput transactionInput = makersDepositTx.getInputs().get(i);
                 depositTx.addInput(getTransactionInput(depositTx, getMakersScriptSigProgram(transactionInput), buyerInputs.get(i)));
+                if (!TransactionWitness.EMPTY.equals(transactionInput.getWitness())) {
+                    depositTx.getInput(depositTx.getInputs().size()-1).setWitness(transactionInput.getWitness());
</pre>
<p>As we do not support now segwit in the trade process, is that change needed at all? I dont see a risk to have it there but maybe better to leave all that out until we start adding segwit support to the trade protocol? Not sure if you started already in more pleaces to add support which do not interfere (as we discussed it is our strategy to add as much as possible without activating segwit in trade protocol...), so if you have started that process already all find, otherwise if its a single change it would be maybe more clear to start after the release with that process...</p>

<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/bisq/pull/4568#pullrequestreview-502272340">view it on GitHub</a>, or <a href="https://github.com/notifications/unsubscribe-auth/AJFFTNTGMPSDYARCWHXFJILSJIHNBANCNFSM4R5D7JSA">unsubscribe</a>.<img src="https://github.com/notifications/beacon/AJFFTNQ7JPINP2FDCYBNMI3SJIHNBA5CNFSM4R5D7JSKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGODXYBCVA.gif" height="1" width="1" alt="" /></p>
<script type="application/ld+json">[
{
"@context": "http://schema.org",
"@type": "EmailMessage",
"potentialAction": {
"@type": "ViewAction",
"target": "https://github.com/bisq-network/bisq/pull/4568#pullrequestreview-502272340",
"url": "https://github.com/bisq-network/bisq/pull/4568#pullrequestreview-502272340",
"name": "View Pull Request"
},
"description": "View this Pull Request on GitHub",
"publisher": {
"@type": "Organization",
"name": "GitHub",
"url": "https://github.com"
}
}
]</script>