{"type":"rich","version":"1.0","title":"Marek Palatinus [ARCHIVE] wrote","author_name":"Marek Palatinus [ARCHIVE] (npub1xt…acz68)","author_url":"https://yabu.me/npub1xtqansnal58rjnnxvyke6kzuvxsz0twg4456awy6ff0525yqgjxqjacz68","provider_name":"njump","provider_url":"https://yabu.me","html":"📅 Original date posted:2018-06-26\n📝 Original message:On Tue, Jun 26, 2018 at 6:58 PM, William Casarin via bitcoin-dev \u003c\nbitcoin-dev at lists.linuxfoundation.org\u003e wrote:\n\u003e\n\u003e seems a bit overkill for how simple the format is, and pulling in a\n\u003e large dependency just for this is a bit silly. Although making it\n\u003e protobuf-compatible is an interesting idea, but I fear would be more\n\u003e work than is worth? I haven't looked closed enough at the protobuf\n\u003e encoding to be sure.\n\u003e\n\u003e \u003e ...while at the same time, implementing \"protobuf-based-BIP174\" by\n\u003e \u003e hand is roughly equally difficult as implementing the current BIP174.\n\u003e\n\u003e as a data point, I was able to build a simple serializer[1] in about an\n\u003e afternoon. I would much prefer to use this lib in say, clightning (my\n\u003e original goal), without having to have the larger protobuf dependency.\n\u003e\n\u003e\nThat was exactly matejcik's point; you can easily write protobuf-compatible\nencoder/decoder for such simple structure in about an afternoon, if you\nneed. Or you can use existing protobuf parsers in matter of minute, if you\ndon't care about dependencies.\n\nAlso, many projects already have protobuf parsers, so it work in other way,\ntoo; you need BIP174 parser as extra dependency/library, although you\nalready use protobuf library (like Trezor device does). For needs of\nBIP174, the difference between ad-hoc format and protobuf is neglible, so\nit is a mistake to introduce yet another format.\n\nslush\n-------------- next part --------------\nAn HTML attachment was scrubbed...\nURL: \u003chttp://lists.linuxfoundation.org/pipermail/bitcoin-dev/attachments/20180626/b277b69a/attachment.html\u003e"}