Merge bitcoin/bitcoin#21824: refactor: [index] Replace deprecated cha…

Merge bitcoin/bitcoin#21824: refactor: [index] Replace deprecated char with uint8_t in serialization

fafb880e8854f9b7fb3934e02a0bd0409aec72c2 refactor: [index] Replace deprecated char with uint8_t in serialization (MarcoFalke)

Pull request description:

  All char representations are serialized in the same way, however the `char` one is deprecated according to https://github.com/bitcoin/bitcoin/blob/d22e7ee93313b13365bd14a5fffeb055cff4dcd2/src/serialize.h#L227 . Also, using `uint8_t` directly avoids casts.

ACKs for top commit:
  jonatack:
    Approach ACK fafb880e8854f9b7fb3934e02a0bd0409aec72c2
  laanwj:
    Code review ACK fafb880e8854f9b7fb3934e02a0bd0409aec72c2
  practicalswift:
    cr ACK fafb880e8854f9b7fb3934e02a0bd0409aec72c2: patch looks correct

Tree-SHA512: ed08fb1b18cb75a695e15924bcaa30ff8746bcd5f17cc83e79f94fe5ff8d9f2083435cb49b8245e3341ede2512140940d864299f4746bc40c8ed8bfdbdacac24

情報元 - Merge bitcoin/bitcoin#21824: refactor: [index] Replace deprecated cha… · bitcoin/[email protected] · GitHub

Merge bitcoin/bitcoin#21824: refactor: [index] Replace deprecated cha… · bitcoin/bitcoin@dc8da2a · GitHub
スポンサーリンク

シェアする

フォローする

スポンサーリンク

トップへ戻る