Name public API structs #1628

pull achow101 wants to merge 1 commits into bitcoin-core:master from achow101:struct-name changing 5 files +12 −12
  1. achow101 commented at 5:57 pm on October 31, 2024: member
    Closes #1627
  2. Name public API structs 694342fdb7
  3. real-or-random added the label feature on Nov 1, 2024
  4. real-or-random added the label refactor/smell on Nov 1, 2024
  5. real-or-random approved
  6. real-or-random commented at 4:07 pm on November 1, 2024: contributor

    utACK 694342fdb71722b31dd4b6bc7241b992fdf6f492

    Fwiw, I had discussed this with @jonasnick because we were not sure if reusing the struct tag as the type name is okay also in C++. But while C++ has slightly different rules here, they’re made so that compatibility to C is ensured, see https://stackoverflow.com/questions/1675351/typedef-struct-vs-struct-definitions

  7. jonasnick approved
  8. jonasnick commented at 4:08 pm on November 1, 2024: contributor
    ACK 694342fdb71722b31dd4b6bc7241b992fdf6f492
  9. real-or-random merged this on Nov 1, 2024
  10. real-or-random closed this on Nov 1, 2024

  11. real-or-random added the label needs-changelog on Nov 1, 2024
  12. real-or-random commented at 4:13 pm on November 1, 2024: contributor
    We may want to add a changelog entry for this PR. The change doesn’t break anything, but it makes using the API more convenient, so users could be interested.
  13. achow101 referenced this in commit 378ca17fd1 on Nov 1, 2024
  14. gatleas17 commented at 8:24 pm on November 2, 2024: none
    ll
  15. achow101 referenced this in commit 2d46a89386 on Nov 4, 2024
  16. vmta referenced this in commit b5ae194ce5 on Nov 6, 2024
  17. vmta referenced this in commit 8999068421 on Nov 6, 2024
  18. Eunovo referenced this in commit 55a2f7a840 on Nov 12, 2024
  19. vmta referenced this in commit f40affbf6c on Nov 21, 2024
  20. vmta referenced this in commit cc8d145633 on Nov 22, 2024
  21. janus referenced this in commit a4b4239cb4 on Jan 19, 2025
  22. div72 referenced this in commit af627d47c3 on Apr 12, 2025
  23. fanquake removed the label needs-changelog on Jul 29, 2025
  24. fanquake commented at 10:28 am on July 29, 2025: member
    Changelog was added in #1631. Dropping “needs-changelog”.

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin-core/secp256k1. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2025-08-30 14:15 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me