Script interpreter cleanups #19875

issue ariard openend this issue on September 4, 2020
  1. ariard commented at 7:58 pm on September 4, 2020: member

    Extracting this topic from #17977 as during its review general cleanups of the script interpreter clean have been advocated.

    Given that taproot implementation review is already far advanced, I think no one is proposing to address them now ? This issue is more to track interesting points raised.

    I guess there is at least two different axis which have been under discussion (but they may overlap a bit):

    If you have more comments worthy to be pinned please add them.

  2. ariard added the label Feature on Sep 4, 2020
  3. JeremyRubin commented at 10:42 pm on September 4, 2020: contributor
    #15969 was a relevant PR where some of this was also discussed.
  4. MarcoFalke commented at 11:55 am on August 3, 2022: member

    The feature request didn’t seem to attract much attention in the past. Also, the issue seems not important enough right now to keep it sitting around idle in the list of open issues.

    Closing due to lack of interest. Pull requests with improvements are always welcome.

  5. MarcoFalke closed this on Aug 3, 2022

  6. ariard commented at 2:21 pm on August 3, 2022: member
    Yeah I still think if it would be nice if we had a board of per-susbsytem (script, mempools, validation, net_processing, etc) refactoring suggestions (maybe in the wiki ?), that could serve as a source of inspirations for any contributors with the patience to do careful refactoring. We’re still loosing a lot of low-level codebase context across the year and I would bet discussions about cleaning up the script interpreter are going to pop up again next time we do substantial modifications there :)
  7. bitcoin locked this on Aug 3, 2023

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-21 18:12 UTC

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