Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add skeleton LLDBParser #424

Open
wants to merge 20 commits into
base: main
Choose a base branch
from

Conversation

ttreyer
Copy link
Contributor

@ttreyer ttreyer commented Dec 11, 2023

Summary

Add the bare-bones class for LLDBParser, with just the public API throwing "unimplemented" exception.
Duplicate the DRGNParser test suite and adapt it for LLDBParser.

Test plan

$ make test

@ttreyer ttreyer requested a review from ajor December 11, 2023 13:09
@ttreyer ttreyer force-pushed the ttreyer/lldb-parser branch 11 times, most recently from 173d1e9 to dcd19a5 Compare December 14, 2023 17:42
@JakeHillion
Copy link
Contributor

This PR has quite a few commits now, would you like to split out the skeleton and land that by itself or wait until it's feature complete? Up to you!

@ttreyer ttreyer force-pushed the ttreyer/lldb-parser branch from dcd19a5 to 0c3dde8 Compare December 19, 2023 18:54
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants