Homebrew/ruby-macho

MachOView needs to support lazy loading for content

rickmark opened this issue · 5 comments

rickmark commented

As we start to parse contents of various regions, it would be helpful to not hold in memory @raw_data but materialize as needed in MachOView.

This can be accomplished by using the new file reference and seek / read to pull in the data lazy

woodruffw commented

Yeah, that's the idea behind #23. I opened that years ago and never got around to it, partially because the performance gains weren't substantial for Homebrew's use case. But I would happily accept a changeset for that now.

rickmark commented
woodruffw commented

Yeah, that's the trouble with mmap. We could consider the mmap gem but in general we've tried to keep ruby-macho at zero dependencies because it gets vendored into Homebrew.

github-actions commented

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

github-actions commented

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.