< achow101> Sometimes (read: usually) I think it was a mistake to add wallet type options as bools to createwallet
< achow101> Should've made that an "options" arg
< luke-jr> yes
< luke-jr> not too late to fix tho?
< achow101> We could add one at the end but that wouldn't necessarily make things better since we still need to keep the existing ones
< luke-jr> could replace the existing ones, and just interpret bools compatibly for a few releases
< achow101> I suppose so
< achow101> Forgot that was a thing we did
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #18991: Cache responses to GETADDR to prevent topology leaks (master...2020-05-addr-response-caching) https://github.com/bitcoin/bitcoin/pull/18991
< bitcoin-git> [bitcoin] MarcoFalke reopened pull request #18991: Cache responses to GETADDR to prevent topology leaks (master...2020-05-addr-response-caching) https://github.com/bitcoin/bitcoin/pull/18991
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #19471: util: Make default arg values more specific (master...200708-hint) https://github.com/bitcoin/bitcoin/pull/19471
< bitcoin-git> [bitcoin] MarcoFalke reopened pull request #19471: util: Make default arg values more specific (master...200708-hint) https://github.com/bitcoin/bitcoin/pull/19471
< bitcoin-git> [bitcoin] hebasto opened pull request #19588: test: Use uniform distribution for random intervals in functional tests (master...200725-uni-test) https://github.com/bitcoin/bitcoin/pull/19588
< instagibbs> unless it's an important improvement I'd be careful about deprecating highly used commands/args combos
< sipa> sdaftuar: i was thinking of tx request data as being a map keyed by (is_wtxid, hash), but perhaps that's not the best mental model
< luke-jr> instagibbs: not highly used if it's that new I'd think
< sipa> and it's better to see is_wtxid more as metadata along with the rest of the data, but just keyed by hash
< sdaftuar> sipa: yes, i think that's basically how i was thinking about it.
< sipa> sdaftuar: that makes sense... it also means m_tx_in_flight and m_tx_announced should probably just be by hash
< sipa> aj: that means i'd be reverting to your original patch, really :)
< aj> sipa: sometimes the simplest approach is the best!