8000 Merge `PtraceDumper` and `MinidumpWriter` · Issue #154 · rust-minidump/minidump-writer · GitHub
[go: up one dir, main page]

Skip to content

Merge PtraceDumper and MinidumpWriter #154

Closed

Description

@marti4d

When this project was first created, it probably made a lot of sense for PtraceDumper and MinidumpWriter to be two different types in case there was ever a "not-ptrace dumper".

At this point, however, the functioning of these two structs is so closely coupled that it's doubtful that real cut lines could actually be made between them without major refactoring work anyway. In the meantime, any time we work on the code there's a weird moment of "Umm... Which struct should this thing be a part of?", and the answer is often "whichever struct already has code that kind-of looks like it".

I ran into that pretty hard when working on #142, where I got annoyed by trying to break my code change up along the seemingly-artificial line between the two. I finally decided to do a table flip and did a bunch of refactoring work. I hope y'all like it!

Activity

  1. self-assigned this
    on Jul 11, 2025
  2. added 2 commits that reference this issue on Jul 11, 2025
    21bb451
    c35af12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

    0