< mryandao> l
< sipa> m
< gmaxwell> f
< luke-jr> k
< bitcoin-git> [bitcoin] steverusso opened pull request #13372: Docs: Linked to 'Building on FreeBSD' in build-unix.md (master...link-to-building-on-freebsd) https://github.com/bitcoin/bitcoin/pull/13372
< wumpus> sipa: the idea to use VARINT for the service bits is that the higher bits are not used, so on the average case this will save some bytes per entry
< wumpus> sipa: and ok, I'm fine with removing the onioncat padding for torv2, it seemed easiest for implementations
< wumpus> sipa: but if you think the VARINT doesn't help I'll change it back to just uint64_t
< sipa> wumpus: my point is more that if you're going to make it variable length... why limit to 64 bits?
< mryandao> has anyone tried to do a `git request-pull` without a github account?
< mryandao> would sending the pending changes to the mailing list prompt any of the maintainers to address the PR?
< wumpus> sipa: my point is more to have a limit, what would you suggest to limit it to?
< wumpus> sipa: my reasoning was: 64 bits is more than enough for now, so there's no actual need to allow longer values
< wumpus> mryandao: that's not really our way of working, I don't think anyone has ever posted a patch to the mailing list
< wumpus> it's even harder to get things to be reviewed that way then on it is on github
< sipa> wumpus: right, but you can have a serialization that supports longer, even though you currently don't use (or accept) it?
< sipa> this is just a nit... i'm totally fine with a varint or just 64 bit, and extend it later again if needed
< wumpus> sipa: right, that'd be ok to me
< wumpus> I've updated the torv2 format to strip the onioncat prefixing
< bitcoin-git> [bitcoin] MarcoFalke closed pull request #12744: WIP: eliminate dependency on boost::program_options (master...program_options) https://github.com/bitcoin/bitcoin/pull/12744
< bitcoin-git> [bitcoin] eudisd opened pull request #13373: Qt: Update Wallet Encryption Titles To Better Describe Process (master...feature/bitcoin-#13245) https://github.com/bitcoin/bitcoin/pull/13373
< bitcoin-git> [bitcoin] kaplanmaxe opened pull request #13374: utils and libraries: checking for bitcoin address in translations (master...check-addr-translations) https://github.com/bitcoin/bitcoin/pull/13374
< bitcoin-git> [bitcoin] undercoverGod opened pull request #13375: utils and libraries: address check in update-translations.py (master...master) https://github.com/bitcoin/bitcoin/pull/13375