Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

feat: Add ConfigManager for handling JSON configs #2

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

cpz
Copy link
Contributor

@cpz cpz commented Mar 15, 2023

This commit message follows the conventional commits format, which starts with a type (e.g., "feat" for feature), followed by a short and concise description of the change. In this case, the message indicates that a new feature, ConfigManager, has been added to handle JSON configuration files.

@cpz cpz requested a review from es3n1n March 15, 2023 23:21
/// <param name="offset">Offset at which the function address is</param>
/// <returns>Address object</returns>
template <typename t = address_base_t<ptr_type>>
inline t jmp(ptrdiff_t offset = 0x1)
Copy link
Collaborator

@es3n1n es3n1n Mar 15, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what if i'm trying to get a virtual address of loc_140000000 from insn, for example:

75 27     jnz     loc_140000000

where displacement value is one-byte-width 27

nlohmann::json config_json_;
};

extern std::unique_ptr<ConfigManager> g_config_manager;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

inline bro

@es3n1n es3n1n marked this pull request as draft August 18, 2023 17:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants